Request: option to provide features not from filesystem

[zhammer]

Hi, I'm curious if y'all would be up allowing a way to provide feature files to godog.TestSuite that is not just passing file paths. Our use case is that we'd like to package a test tool as a binary using godog and ship features as static files in the binary using https://github.com/markbates/pkger. Here we'd have he feature file bodies and names; however, the TestSuite wouldn't be able to parse them from a list of filesystem Paths.

Happy to discuss different ways this could work!

godog/options.go

Lines 54 to 55 in 28ad994

	// All feature file paths
	Paths []string

[lonnblad]

Sounds interesting, maybe when building the godog testrunner we could have an include functionality that would embed the feature dirs/files you pass as arguments.

Something like: > godog build --include --output godog.test features would build a test runner binary which embeds the features dir.

[zhammer]

Yep that’d be nice. We have our own golang cli that uses the godog package directly so wouldn’t immediately need the cli option, but I do like the idea of including packaged feature files in the godog cli tool itself

[lonnblad]

Were you thinking something like:

type GherkinFile struct {
  Name    string
  Content []byte
}

type Options struct {
  ...
  Files []GherkinFile
}

and let your code read the content from pkger?

[lcarrazana]

Looking for the exact same functionality.
I'm using pkger for embedding all static files, and would like to also do it for godog feature files.

[lonnblad]

@lcarrazana would you want to use it in a similar way as zhammer, or would you use the godog CLI directly?

[vearutop]

Now that Go provides standard embed package, this feature might be especially interesting for building portable test runners with embedded test scenarios.

We can follow the approach used in https://github.com/pressly/goose/pull/244/files (setting base fs of runner and keeping paths) to implement such feature with minimal API changes.

[vearutop]

Done with #476.