✨ Drop command

cmd/drop.go

@@ -17,36 +17,76 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
 package cmd
 
 import (
+	"encoding/json"
 	"fmt"
+	"time"
 
+	"codeberg.org/danjones000/my-log/files"
+	"codeberg.org/danjones000/my-log/models"
+	"codeberg.org/danjones000/my-log/tools"
 	"github.com/spf13/cobra"
 )
 
+var dateStr string
+var fields map[string]string
+var j Json
+
 // dropCmd represents the drop command
 var dropCmd = &cobra.Command{
-	Use:   "drop",
-	Short: "A brief description of your command",
-	Long: `A longer description that spans multiple lines and likely contains examples
-and usage of using your command. For example:
-
-Cobra is a CLI library for Go that empowers applications.
-This application is a tool to generate the needed files
-to quickly create a Cobra application.`,
-	Run: func(cmd *cobra.Command, args []string) {
-		fmt.Println("drop called")
+	Use:   "drop log title",
+	Short: "Add a new log entry",
+	// Long: ``,
+	Args:         cobra.ExactArgs(2),
+	SilenceUsage: true,
+	RunE: func(cmd *cobra.Command, args []string) error {
+		log := args[0]
+		title := args[1]
+		e := models.PartialEntry()
+		if len(j.RawMessage) > 8 {
+			err := json.Unmarshal([]byte(j.RawMessage), &e)
+			if err != nil {
+				return err
+			}
+		}
+		for k, v := range fields {
+			e.Fields = append(e.Fields, models.Meta{k, tools.ParseString(v)})
+		}
+		e.Title = title
+		e.Date = time.Now().Local() // @todo parse date
+		l := models.Log{log, []models.Entry{e}}
+		err := files.Append(l)
+		if err != nil {
+			return err
+		}
+		by, err := e.MarshalText()
+		if err != nil {
+			return err
+		}
+		fmt.Fprintf(cmd.OutOrStdout(), "%s\n", by)
+		return nil
 	},
 }
 
 func init() {
 	rootCmd.AddCommand(dropCmd)
 
-	// Here you will define your flags and configuration settings.
-
-	// Cobra supports Persistent Flags which will work for this command
-	// and all subcommands, e.g.:
-	// dropCmd.PersistentFlags().String("foo", "", "A help for foo")
-
-	// Cobra supports local flags which will only run when this command
-	// is called directly, e.g.:
-	// dropCmd.Flags().BoolP("toggle", "t", false, "Help message for toggle")
+	dropCmd.Flags().StringVarP(&dateStr, "date", "d", time.Now().Local().Format(time.RFC3339), "Date for log entry")
+	dropCmd.Flags().StringToStringVarP(&fields, "fields", "f", nil, "Fields you add to entry")
+	dropCmd.Flags().VarP(&j, "json", "j", "Entire entry as json")
+}
+
+type Json struct {
+	json.RawMessage
+}
+
+func (j *Json) String() string {
+	return string(j.RawMessage)
+}
+
+func (j *Json) Set(in string) error {
+	return json.Unmarshal([]byte(in), &j.RawMessage)
+}
+
+func (j *Json) Type() string {
+	return "json"
 }

config/load.go

@@ -14,7 +14,7 @@ import (
 )
 
 var ConfigPath string
-var Overrides map[string]string
+var Overrides = map[string]string{}
 
 func init() {
 	conf, _ := os.UserConfigDir()

files/append.go

@@ -0,0 +1,42 @@
+package files
+
+import (
+	"fmt"
+	"os"
+	fp "path/filepath"
+	"strings"
+
+	"codeberg.org/danjones000/my-log/config"
+	"codeberg.org/danjones000/my-log/models"
+)
+
+func Append(l models.Log) error {
+	conf, err := config.Load()
+	if err != nil {
+		return err
+	}
+
+	filename := fmt.Sprintf("%s.%s", strings.ReplaceAll(l.Name, ".", string(os.PathSeparator)), conf.Input.Ext)
+	path := fp.Join(conf.Input.Path, filename)
+	dir := fp.Dir(path)
+	err = os.MkdirAll(dir, 0750)
+	if err != nil {
+		return err
+	}
+
+	f, err := os.OpenFile(path, os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0640)
+	if err != nil {
+		return err
+	}
+	defer f.Close()
+
+	for _, e := range l.Entries {
+		by, err := e.MarshalText()
+		if err != nil {
+			continue
+		}
+		f.Write(by)
+	}
+
+	return nil
+}

files/append_test.go

@@ -0,0 +1,91 @@
+package files
+
+import (
+	"fmt"
+	"os"
+	"testing"
+	"time"
+
+	"codeberg.org/danjones000/my-log/config"
+	"codeberg.org/danjones000/my-log/models"
+	"github.com/stretchr/testify/suite"
+)
+
+func TestAppend(t *testing.T) {
+	suite.Run(t, new(AppendTestSuite))
+}
+
+type AppendTestSuite struct {
+	suite.Suite
+	dir string
+}
+
+func (s *AppendTestSuite) SetupSuite() {
+	s.dir, _ = os.MkdirTemp("", "append-test")
+	config.Overrides["input.path"] = s.dir
+	config.Overrides["input.ext"] = "log"
+}
+
+func (s *AppendTestSuite) TearDownSuite() {
+	os.RemoveAll(s.dir)
+	delete(config.Overrides, "input.path")
+	delete(config.Overrides, "input.ext")
+}
+
+func (s *AppendTestSuite) TestSuccess() {
+	when := time.Now().Local()
+	e := models.Entry{
+		Title: "Jimmy",
+		Date:  when,
+		Fields: []models.Meta{
+			{"foo", 42},
+			{"bar", true},
+		},
+	}
+	l := models.Log{
+		Name:    "test",
+		Entries: []models.Entry{e},
+	}
+	err := Append(l)
+	s.Require().NoError(err)
+	s.Require().FileExists(s.dir + "/test.log")
+	// @todo test file contents
+}
+
+func (s *AppendTestSuite) TestConfLoadErr() {
+	currConf := config.ConfigPath
+	tmp, _ := os.CreateTemp("", "app-conf-*.toml")
+	fname := tmp.Name()
+	defer tmp.Close()
+	defer os.Remove(fname)
+	fmt.Fprintln(tmp, `{"not":"toml"}`)
+	config.ConfigPath = fname
+	defer func(path string) {
+		config.ConfigPath = path
+	}(currConf)
+	err := Append(models.Log{})
+	s.Assert().ErrorContains(err, "toml")
+}
+
+func (s *AppendTestSuite) TestMkdirErr() {
+	// Don't run this test as root
+	config.Overrides["input.path"] = "/root/my-logs-test"
+	defer func(path string) {
+		config.Overrides["input.path"] = path
+	}(s.dir)
+	err := Append(models.Log{})
+	s.Assert().ErrorContains(err, "permission denied")
+}
+
+func (s *AppendTestSuite) TestOpenErr() {
+	l := models.Log{
+		Name: "test-open-err",
+	}
+	fname := s.dir + "/test-open-err.log"
+	os.MkdirAll(s.dir, 0750)
+	f, _ := os.Create(fname)
+	f.Close()
+	os.Chmod(fname, 0400) // read only
+	err := Append(l)
+	s.Assert().ErrorContains(err, "permission denied")
+}

models/entry.go

@@ -20,6 +20,10 @@ type Entry struct {
 	skipMissing bool
 }
 
+func PartialEntry() Entry {
+	return Entry{skipMissing: true}
+}
+
 type metaRes struct {
 	out []byte
 	err error
@@ -68,7 +72,7 @@ func (e Entry) MarshalText() ([]byte, error) {
 	}
 	ch := e.getFieldMarshalChan()
 	buff := &bytes.Buffer{}
-	buff.WriteString("@begin ")
+	buff.WriteString("\n@begin ")
 	buff.WriteString(e.Date.Format(DateFormat))
 	buff.WriteString(" - ")
 	buff.WriteString(e.Title)

models/entry_test.go

@@ -9,6 +9,7 @@ import (
 	"time"
 
 	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 // Type assertions
@@ -17,6 +18,19 @@ var _ encoding.TextUnmarshaler = new(Entry)
 var _ json.Marshaler = Entry{}
 var _ json.Unmarshaler = new(Entry)
 
+func TestPartialEntry(t *testing.T) {
+	e := PartialEntry()
+	assert.True(t, e.skipMissing)
+	err := json.Unmarshal([]byte(`{"a":42}`), &e)
+	assert.NoError(t, err)
+	assert.Equal(t, "", e.Title)
+	assert.Equal(t, time.Time{}, e.Date)
+	require.Len(t, e.Fields, 1)
+	f := e.Fields[0]
+	assert.Equal(t, "a", f.Key)
+	assert.Equal(t, int64(42), f.Value)
+}
+
 func TestEntryMarshal(t *testing.T) {
 	when := time.Now()
 	whens := when.Format(DateFormat)
@@ -77,12 +91,12 @@ func getEntryMarshalTestRunner(title string, date time.Time, fields []Meta, firs
 			return
 		}
 		if len(lines) == 0 {
-			assert.Equal(t, first, string(o))
+			assert.Equal(t, "\n"+first, string(o))
 			return
 		}
 
 		os := string(o)
-		assert.Regexp(t, "^"+first, os)
+		assert.Regexp(t, "^\n"+first, os)
 		for _, line := range lines {
 			assert.Regexp(t, "(?m)^"+line, os)
 		}