-
Notifications
You must be signed in to change notification settings - Fork 7
Delete participants automated test (plus other study related tests) #6732
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
Conversation
study/src/org/labkey/study/model/ParticipantIdImportHelper.java
Outdated
Show resolved
Hide resolved
private void createDataset(Connection conn, String kindName, String datasetName, String folderName) throws Exception | ||
{ | ||
CreateDomainCommand cmd = new CreateDomainCommand(kindName, datasetName); | ||
|
||
cmd.getDomainDesign().setFields(List.of( | ||
new FieldDefinition("Name", FieldDefinition.ColumnType.String), | ||
new FieldDefinition("IntField", FieldDefinition.ColumnType.Integer), | ||
new FieldDefinition("DoubleField", FieldDefinition.ColumnType.Double) | ||
)); | ||
if ("Demographics".equalsIgnoreCase(datasetName)) | ||
cmd.setOptions(Map.of("demographics", true)); | ||
cmd.execute(conn, folderName); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider making a DomainProps
subclass for datasets (similar to ListDefinition
or SampleTypeDefinition
)
Might be useful when we start adding new SDMS tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I was not aware of those classes. I'll take a look.
Rationale
Adding automation for recent feature or bug fixes:
StudyManageParticipantsTest
which was renamed from :StudyMergeParticipantsTest
.