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

Failing test TestEPCISWSDL #253

Open
sblackstone opened this issue Jun 8, 2023 · 2 comments
Open

Failing test TestEPCISWSDL #253

sblackstone opened this issue Jun 8, 2023 · 2 comments

Comments

@sblackstone
Copy link

sblackstone commented Jun 8, 2023

Using Go 1.20

It seems like its looking for an exact match to a sample file, but the output has changed slightly...

image

It's not clear to me what this is attempting to test or how to resolve it....

@sblackstone
Copy link
Author

The test output is this:

--- FAIL: TestEPCISWSDL (0.01s)
    gowsdl_test.go:320: got source ./fixtures/epcis/epcisquery_gen.src but expected ./fixtures/epcis/epcisquery.src

@jordanweschler
Copy link

I went through with git bisect and found the original commit that broke that test to be 9a29f8e . This is problematic, because that commit created that test as well. So it seems it was never passing.

3aaab23 has the message "fix broken tests" but the tests still fail on that commit despite the adjustments made.

From looking at the content of the WSDL, the current generated source epciquery_gen.src should be correct; the core issue being the xml.Name member being added, which reflects the structure of the WSDL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants