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

Subcolumns must be ordered #15

Closed
tcmj opened this issue Jul 18, 2017 · 2 comments
Closed

Subcolumns must be ordered #15

tcmj opened this issue Jul 18, 2017 · 2 comments
Labels
bug Error
Milestone

Comments

@tcmj
Copy link
Owner

tcmj commented Jul 18, 2017

if you define subcolumns the order bust be sorted
eg. 3,4,5 works
4,5,3 does not work

	at com.tcmj.pug.enums.builder.impl.JavaPoetEnumBuilder.build(JavaPoetEnumBuilder.java:80)
	at com.tcmj.pug.enums.api.fluent.Fluent.build(Fluent.java:151)
	at com.tcmj.pug.enums.example.fluent.WikipediaExample3.main(WikipediaExample3.java:28)
Caused by: java.lang.IllegalArgumentException: unused arguments: expected 2, received 3
	at com.squareup.javapoet.Util.checkArgument(Util.java:64)
	at com.squareup.javapoet.CodeBlock$Builder.add(CodeBlock.java:243)
	at com.squareup.javapoet.TypeSpec.anonymousClassBuilder(TypeSpec.java:115)
	at com.tcmj.pug.enums.builder.impl.JavaPoetEnumBuilder.writeEnumConstants(JavaPoetEnumBuilder.java:156)
	at com.tcmj.pug.enums.builder.impl.JavaPoetEnumBuilder.build(JavaPoetEnumBuilder.java:45)```
@tcmj tcmj added the bug Error label Jul 18, 2017
@tcmj
Copy link
Owner Author

tcmj commented Jul 18, 2017

possible cause might be in this method: isColumnInArray of class URLHtmlDataProvider ??

@tcmj tcmj added this to Ready in sprint_1_2_7_15 Jul 18, 2017
@tcmj tcmj moved this from Ready to InProgress in sprint_1_2_7_15 Jul 18, 2017
@tcmj tcmj added this to the 1.2.7.15 milestone Jul 18, 2017
@tcmj
Copy link
Owner Author

tcmj commented Jul 18, 2017

We start by implementing some unit tests which describes the expected behavour

@tcmj tcmj closed this as completed in d9d5038 Jul 18, 2017
@tcmj tcmj moved this from InProgress to Verification in sprint_1_2_7_15 Jul 18, 2017
@tcmj tcmj moved this from Verification to Finished in sprint_1_2_7_15 Jul 18, 2017
@tcmj tcmj added this to open in sprint_1_2_9_15 Sep 5, 2017
@tcmj tcmj removed this from open in sprint_1_2_9_15 Sep 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Error
Projects
No open projects
Development

No branches or pull requests

1 participant