diff --git a/app/cmd/serve.go b/app/cmd/serve.go index 40e1768d..4dcfa214 100644 --- a/app/cmd/serve.go +++ b/app/cmd/serve.go @@ -4,9 +4,6 @@ import ( "fmt" "os" - "github.com/go-logr/zapr" - "go.uber.org/zap" - "github.com/jlewi/foyle/app/pkg/application" "github.com/jlewi/monogo/helpers" @@ -36,8 +33,6 @@ func NewServeCmd() *cobra.Command { defer helpers.DeferIgnoreError(app.Shutdown) logVersion() - log := zapr.NewLogger(zap.L()) - log.Info("Starting server") return s.Run() }() diff --git a/app/pkg/application/app.go b/app/pkg/application/app.go index 656eac8b..74cdd6a9 100644 --- a/app/pkg/application/app.go +++ b/app/pkg/application/app.go @@ -294,14 +294,5 @@ func (a *App) Shutdown() error { for _, closer := range a.logClosers { closer() } - //if err := l.Sync(); err != nil { - // // N.B. I don't understand why but calling sync appears to cause an error complaining about calling sync - // // on /dev/stderr so we just filter that out to avoid spam. - // pathErr := &fs.PathError{} - // - // if !errors.As(err, &pathErr) { - // return err - // } - //} return nil } diff --git a/frontend/foyle/src/web/client.ts b/frontend/foyle/src/web/client.ts index d7f1cfc3..19e23386 100644 --- a/frontend/foyle/src/web/client.ts +++ b/frontend/foyle/src/web/client.ts @@ -15,8 +15,7 @@ export class FoyleClient { // Include a default so that address is always well defined const address = config.get("executor-address", "http://localhost:8080"); - const body = request.toJsonString(); - console.log(`Sending request ${body}`); + const body = request.toJsonString(); return fetch(address + "/api/v1alpha1/execute", { method: 'POST', body: body, @@ -43,7 +42,6 @@ export class FoyleClient { }) .then(response => response.json()) .then(data => { - console.log(`Generate response ${data}`); const resp = agentpb.GenerateResponse.fromJson(data); return resp; }); diff --git a/frontend/foyle/src/web/test/suite/metadata.test.ts b/frontend/foyle/src/web/test/suite/metadata.test.ts index bc581d4e..eb7a83f7 100644 --- a/frontend/foyle/src/web/test/suite/metadata.test.ts +++ b/frontend/foyle/src/web/test/suite/metadata.test.ts @@ -41,18 +41,6 @@ suite('Metadata Test Suite', () => { } }); - test('test setBlockFromMeta-undefined', () => { - // This test case was trying to reproduce the error in the browser that - // "meta.hasOwnProperty is not a function" - - // No traceids - const meta1 = Object.create(null); - const actual = new docpb.Block(); - metadata.setBlockFromMeta(actual, meta1); - - - }); - test('test setBlockFromMeta', () => { class TestCase { meta: metadata.CellMetadata;