Go test only checks main_test.go about nx-go HOT 9 CLOSED

I think this needs to change from this:

export function runBuilder(options: BuildBuilderSchema, context: BuilderContext): Observable<BuilderOutput> {
  return from(context.getProjectMetadata(context?.target?.project)).pipe(
    map((project) => {
      const root = project.root
      const sources = `-v ${root}/**/*.go`

      return runGoCommand(context, 'test', [sources])
    }),
  )
}

to this:

export function runBuilder(options: BuildBuilderSchema, context: BuilderContext): Observable<BuilderOutput> {
  return from(context.getProjectMetadata(context?.target?.project)).pipe(
    map((project) => {
       const sources = `./... -v`; // also could include -race by default
        return runGoCommand(context, 'test', [sources], { cwd: project.root});
    }),
  )
}

from nx-go.

Also forgot about this, in the workspace.json it the builder needs to change from:

"test": {
  "builder": "@nx-go/nx-go:test",
  "options": {
    "main": "apps/api/main_test.go"
  }
}

to just:

"test": {
  "builder": "@nx-go/nx-go:test"
}

from nx-go.

@beeman let me know if you want me to PR on this

Hey @Narven - I would really appreciate a PR! I should have some time this weekend en next weekend to look at nx-go!

I'm about to start a pet project that uses Go so I'll be using this again!

from nx-go.

The best approach it think, it would be to allow any extra arg be passed, that would allowd to had for example -race or -cover

from nx-go.

@beeman let me know if you want me to PR on this

from nx-go.

@beeman I do not have permissions to push a new branch

from nx-go.

@Narven you can create a Pr by forking this repo and creating a branch on your fork. Then you should be able to open a PR here.

from nx-go.

This is now fixed in version 1.3.0

from nx-go.

The best approach it think, it would be to allow any extra arg be passed, that would allowd to had for example -race or -cover

Thanks for the suggestion! These are available and enabled by default in 1.4.0.

from nx-go.