HGKeeper

Port from logrus to log/slog
default tip
5 months ago, Elliott Sales de Andrade

c588f9b3f559

Port from logrus to log/slog

This doesn't really take much advantage of structured logging beyond what is already done (`id` and `idColor`), and consequently the log handler does not try to do any handling of anything more than that (i.e., grouping, or arbitrary attributes beyond those defined).

One should maybe have a `Context` available to pass in, but there isn't one, and anyway, the log handler doesn't use it, so I've passed in a `TODO` instead.

Everything else is just normal import/rename changes.

Testing Done:
Ran `go run . run`

Reviewed at https://reviews.imfreedom.org/r/2871/

// Convey

// This program is free software: you can redistribute it and/or modify

// it under the terms of the GNU General Public License as published by

// the Free Software Foundation, either version 3 of the License, or

// (at your option) any later version.

// This program is distributed in the hope that it will be useful,

// but WITHOUT ANY WARRANTY; without even the implied warranty of

// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the

// GNU General Public License for more details.

// You should have received a copy of the GNU General Public License

// along with this program. If not, see <http://www.gnu.org/licenses/>.

package tasks

import (

"os"

"path/filepath"

"testing"

"time"

"github.com/stretchr/testify/assert"

"keep.imfreedom.org/grim/convey/environment"

"keep.imfreedom.org/grim/convey/fs"

"keep.imfreedom.org/grim/convey/logging"

"keep.imfreedom.org/grim/convey/runtime"

)

func testImport(t *testing.T, files []string) {

env := environment.New()

// create our runtime

rt := runtime.New(environment.New(), "", environment.New(), false, false, 0*time.Second)

defer rt.Shutdown()

// create our task and run it.

importTask := &Import{Files: files}

err := importTask.Execute("", logging.NewAdapter("test"), env, rt)

assert.Nil(t, err)

for _, file := range files {

spec := fs.ParsePathSpec(file)

ws, err := rt.Workspace()

assert.Nil(t, err)

_, err = os.Stat(filepath.Join(ws.Path(), spec.Destination()))

assert.NotEqual(t, err, os.IsNotExist(err))

}

// TestImportSingleSourceOnly tests that the following yaml functions

// correctly:

// single-source-only:

// type: convey/import

// files:

// - import_test.go

func TestImportSingleSourceOnly(t *testing.T) {

testImport(t, []string{"import_test.go"})

}

// TestImportSingleSourceWithDestination tests that the following yaml

// functions correctly:

// single-source-with-destination:

// type: convey/import

// files:

// - import_test.go:imported.go

func TestImportSingleSourceWithDestination(t *testing.T) {

testImport(t, []string{"import_test.go:imported.go"})

}