Skip to content

Conversation

@j-sal
Copy link
Collaborator

@j-sal j-sal commented Oct 20, 2022

Fixes #56

@j-sal j-sal force-pushed the 56-reconcile-filenames branch from 67752ab to dbffc78 Compare October 20, 2022 11:26
rows.add(frtr.getNextRowOfCells());

Assert.assertEquals(rows.get(0), Arrays.asList("File:LasTres.jpg", "M127722", "Category:Costa Rica"));
Assert.assertEquals(rows.get(0).toString(), Arrays.asList("File:LasTres.jpg, M127722, Category:Costa Rica").toString());
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here you could also try to check that the Recon object is present as well (but not a big deal without)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This check has been implemented in PR #77

Copy link
Member

@wetneb wetneb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

@wetneb wetneb merged commit 9c1e0de into OpenRefine:master Oct 20, 2022
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

Successfully merging this pull request may close these issues.

Filenames already reconciled when starting an OpenRefine project from Commons categories

2 participants