Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make test fails when gotestsum is installed #705

Open
cuminandpaprika opened this issue Mar 18, 2020 · 0 comments
Open

make test fails when gotestsum is installed #705

cuminandpaprika opened this issue Mar 18, 2020 · 0 comments
Assignees
Labels
P2 Low priority

Comments

@cuminandpaprika
Copy link
Contributor

Description

running make test fails when gotestsum is installed.

Further investigation shows that running go test --json ./... produces a failing output:

{"Time":"2020-03-18T16:56:08.651278+11:00","Action":"fail","Package":"github.com/anz-bank/sysl/cmd/sysl","Test":"TestMain3","Elapsed":0.5}

Steps to Reproduce

  1. Checkout master branch
  2. run make test with gotestsum installed

Expected behavior:
All tests should pass as running go test ./... passes

Actual behavior:
Test fails, output shown below

  • Debug&Crash Output
    Using gotestsum installed via go get gotest.tools/gotestsum
    go version go1.13.8 darwin/amd64 from commit hash: gotestyourself/gotestsum@b9c9035
=== Failed
=== FAIL: cmd/sysl TestMain3 (0.54s)
usage: sysl [<flags>] <command> [<args> ...]
System Modelling Language Toolkit
Optional flags:
      --help        Show context-sensitive help (also try --help-long and
                    --help-man).
      --version     Show application version.
      --log="warn"  log level: [trace,off,debug,info,warn]
  -v, --verbose     enable verbose logging
      --root=ROOT   sysl root directory for input model file. If root is not
                    found, the module directory becomes the root, but the module
                    can not import with absolute paths (or imports must be
                    relative).
Commands:
  help [<command>...]
    Show help.
  codegen --transform=TRANSFORM --grammar=GRAMMAR [<flags>] <MODULE>...
    Generate code
  datamodel [<flags>] <MODULE>...
    Generate data models
  env
    Print sysl environment information.
  export [<flags>] <MODULE>...
    Export sysl to external types. Supported types: Swagger
  generate-db-scripts [<flags>] <MODULE>...
    Generate db script
  generate-db-scripts-delta [<flags>] <MODULE>...
    Generate delta db scripts
  import --input=INPUT --app-name=APP-NAME [<flags>]
    Import foreign type to sysl. Supported types: [grammar, openapi, swagger,
    xsd]
  info
    Show binary information
  integrations [<flags>] <MODULE>...
    Generate integrations
  mod init [<name>]
    initializes and writes a new go.mod to the current directory
  protobuf [<flags>] <MODULE>...
    Generate textpb/json
  repl
    Enter a sysl REPL
  sd [<flags>] <MODULE>...
    Generate Sequence Diagram
  template --template=TEMPLATE --start=START [<flags>] <MODULE>...
    Apply a model to a template for custom text output
  test-rig --template=TEMPLATE [<flags>] <MODULE>...
    Generate test rig
  ui [<flags>] <MODULE>...
    Starts the Sysl UI which displays a visual view of Apps defined in the sysl
    model.
  validate <MODULE>...
    Validate the sysl file
ok  	github.com/anz-bank/sysl/cmd/sysl	0.526s
DONE 979 tests, 1 failure in 6.255s
@cuminandpaprika cuminandpaprika self-assigned this Mar 18, 2020
cuminandpaprika referenced this issue Mar 19, 2020
* #619 another approach to fix this issue, its command line will looks like 'sysl codegen @config_file' and config_file has the all of command flags. Actually this approach can simply all commands.

* #619 update test data and test case.
@ChloePlanet ChloePlanet added the P2 Low priority label Apr 3, 2020
anzopensource pushed a commit that referenced this issue Jul 4, 2022
* Add TypeScript representation of Sysl model

* Add publish workflow

* WIP version 3 of model

* Exclude elements and statements from model

* Setup node in test workflow

* Update registry url

* Address feedback

* Fix makefile

* Fix rendering indentation

* Provide sysl binary for TS tests

* Delete yarn.lock

* Fix npm package name

Co-authored-by: Oliver <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P2 Low priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants