Skip to content

Commit 1790087

Browse files
authored
Fix data region column name references (#168)
1 parent 23dbaa4 commit 1790087

File tree

2 files changed

+25
-25
lines changed

2 files changed

+25
-25
lines changed

OConnorExperiments/test/src/org/labkey/test/tests/OConnorExperimentTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ protected void verifyExperimentWebpart(int row, String description, @Nullable St
146146
assertEquals(description, table.getDataAsText(row, "Description"));
147147
if (type != null)
148148
{
149-
assertEquals(type, table.getDataAsText(row, "ExperimentType"));
149+
assertEquals(type, table.getDataAsText(row, "ExperimentTypeId"));
150150
}
151151

152152
// Make sure each component of the ParentExperiments column is rendered with a link to the begin page for that experiment

genotyping/test/src/org/labkey/test/tests/HaplotypeAssayTest.java

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -160,10 +160,10 @@ private void verifyAribitraryHaplotypeAssay()
160160
clickAndWait(Locator.linkWithText(DRB_RUN));
161161
DataRegionTable drt = new DataRegionTable("Data", this);
162162

163-
verifyColumnDataValues(drt, "MHC-AHaplotype1", "A001", "A023", "A001", "A004", "A002a");
164-
verifyColumnDataValues(drt, "MHC-AHaplotype2", "A023", "A021", "A001", "A023", "A002a");
165-
verifyColumnDataValues(drt, "MHC-DRB Haplotype 1", "D025a", "D012b", "D001c", "D012b", "D002");
166-
verifyColumnDataValues(drt, "MHC-DRB Haplotype 2", "D015c", "D017a", "D017a", "D012b", "D002");
163+
verifyColumnDataValues(drt, "mhcAHaplotype1", "A001", "A023", "A001", "A004", "A002a");
164+
verifyColumnDataValues(drt, "mhcAHaplotype2", "A023", "A021", "A001", "A023", "A002a");
165+
verifyColumnDataValues(drt, "mhcDRBHaplotype1", "D025a", "D012b", "D001c", "D012b", "D002");
166+
verifyColumnDataValues(drt, "mhcDRBHaplotype2", "D015c", "D017a", "D017a", "D012b", "D002");
167167

168168
importRun(STR_RUN, DRB_ASSAY, STR_RUN_FILE, true);
169169

@@ -286,13 +286,13 @@ private void verifyFirstRun()
286286
verifyColumnDataValues(drt, "Animal", "ID-1", "ID-2", "ID-3", "ID-4", "ID-5");
287287
verifyColumnDataValues(drt, "TotalReads", "1000", "2000", "3000", "4000", "5000");
288288
verifyColumnDataValues(drt, "IdentifiedReads", "300", "1000", "600", "2500", "3250");
289-
verifyColumnDataValues(drt, "%Unknown", "70.0", "50.0", "80.0", "37.5", "35.0");
290-
verifyColumnDataValues(drt, "MHC-AHaplotype1", "A001", "A023", "A001", "A004", "A002a");
291-
verifyColumnDataValues(drt, "MHC-AHaplotype2", "A023", "A021", "A001", "A023", "A002a");
292-
verifyColumnDataValues(drt, "MHC-BHaplotype1", "B015c", "B012b", "B001c", "B012b", "B002");
293-
verifyColumnDataValues(drt, "MHC-BHaplotype2", "B025a", "B017a", "B017a", "B012b", "B002");
289+
verifyColumnDataValues(drt, "PercentUnknown", "70.0", "50.0", "80.0", "37.5", "35.0");
290+
verifyColumnDataValues(drt, "mhcAHaplotype1", "A001", "A023", "A001", "A004", "A002a");
291+
verifyColumnDataValues(drt, "mhcAHaplotype2", "A023", "A021", "A001", "A023", "A002a");
292+
verifyColumnDataValues(drt, "mhcBHaplotype1", "B015c", "B012b", "B001c", "B012b", "B002");
293+
verifyColumnDataValues(drt, "mhcBHaplotype2", "B025a", "B017a", "B017a", "B012b", "B002");
294294
verifyColumnDataValues(drt, "Enabled", "true", "true", "true", "true", "true");
295-
verifyColumnDataValues(drt, "ClientAnimalId", "x123", "x234", "x345", "x456", "x567");
295+
verifyColumnDataValues(drt, "AnimalId/ClientAnimalId", "x123", "x234", "x345", "x456", "x567");
296296

297297
// verify concatenated haplotype strings
298298
List<String> concatenated = drt.getColumnDataAsText("ConcatenatedHaplotypes");
@@ -307,8 +307,8 @@ private void verifyFirstRun()
307307
goToQuery("Animal");
308308
drt = new DataRegionTable("query", this);
309309
assertEquals("Unexpected number of Animal records", 5, drt.getDataRowCount());
310-
verifyColumnDataValues(drt, "Lab Animal Id", "ID-1", "ID-2", "ID-3", "ID-4", "ID-5");
311-
verifyColumnDataValues(drt, "Client Animal Id", "x123", "x234", "x345", "x456", "x567");
310+
verifyColumnDataValues(drt, "LabAnimalId", "ID-1", "ID-2", "ID-3", "ID-4", "ID-5");
311+
verifyColumnDataValues(drt, "ClientAnimalId", "x123", "x234", "x345", "x456", "x567");
312312

313313
verifyHaplotypeRecordsByType(11, 5, 6);
314314
}
@@ -325,7 +325,7 @@ private void verifySecondRun()
325325

326326
// validate extra column in view
327327
DataRegionTable drt = new DataRegionTable("Data", this);
328-
verifyColumnDataValues(drt, "ClientAnimalId", "x456", "x567", "x678", "x789", "x888", "x999");
328+
verifyColumnDataValues(drt, "AnimalId/ClientAnimalId", "x456", "x567", "x678", "x789", "x888", "x999");
329329

330330
// verify that the animal and haplotype rows were properly inserted
331331
goToQuery("Animal");
@@ -344,11 +344,11 @@ private void verifySecondRun(String prefix)
344344
verifyColumnDataValues(drt, "Animal", "ID-4", "ID-5", "ID-6", "ID-7", "ID-8", "ID-9");
345345
verifyColumnDataValues(drt, "TotalReads", "4000", "5000", "6000", "7000", " ", "0");
346346
verifyColumnDataValues(drt, "IdentifiedReads", "2500", "3250", "3000", "3500", " ", "1");
347-
verifyColumnDataValues(drt, "%Unknown", "37.5", "35.0", "50.0", "50.0", " ", " ");
348-
verifyColumnDataValues(drt, prefix+"-AHaplotype1", "A001", " ", "A033", "A004", "A004", "A004");
349-
verifyColumnDataValues(drt, prefix+"-AHaplotype2", "A023", " ", "A033", " ", "A004", "A004");
350-
verifyColumnDataValues(drt, prefix+"-BHaplotype1", "B015c", " ", "B012b", "B033", "B033", "B033");
351-
verifyColumnDataValues(drt, prefix+"-BHaplotype2", "B025a", " ", "B012b", "B033", "B033", "B033");
347+
verifyColumnDataValues(drt, "PercentUnknown", "37.5", "35.0", "50.0", "50.0", " ", " ");
348+
verifyColumnDataValues(drt, prefix+"AHaplotype1", "A001", " ", "A033", "A004", "A004", "A004");
349+
verifyColumnDataValues(drt, prefix+"AHaplotype2", "A023", " ", "A033", " ", "A004", "A004");
350+
verifyColumnDataValues(drt, prefix+"BHaplotype1", "B015c", " ", "B012b", "B033", "B033", "B033");
351+
verifyColumnDataValues(drt, prefix+"BHaplotype2", "B025a", " ", "B012b", "B033", "B033", "B033");
352352
verifyColumnDataValues(drt, "Enabled", "true", "true", "true", "true", "true", "true");
353353

354354
// verify concatenated haplotype strings
@@ -398,14 +398,14 @@ private void verifyExtraHaplotypeAssignment()
398398
waitForElement(Locator.paginationText(1, 39, 39));
399399

400400
// ADD: animal ID-5, haplotype A001
401-
DataRegionTable.findDataRegion(this).clickInsertNewRow();
401+
DataRegionTable.DataRegion(getDriver()).find().clickInsertNewRow();
402402
selectOptionByText(Locator.name("quf_HaplotypeId"), "A001");
403403
selectOptionByText(Locator.name("quf_AnimalAnalysisId"), animalAnalysisId);
404404
setFormElement(Locator.name("quf_DiploidNumber"), "1");
405405
clickButton("Submit");
406406

407407
// ADD: animal ID-5, haplotype B002
408-
DataRegionTable.findDataRegion(this).clickInsertNewRow();
408+
DataRegionTable.DataRegion(getDriver()).find().clickInsertNewRow();
409409
selectOptionByText(Locator.name("quf_HaplotypeId"), "B002");
410410
selectOptionByText(Locator.name("quf_AnimalAnalysisId"), animalAnalysisId);
411411
setFormElement(Locator.name("quf_DiploidNumber"), "1");
@@ -416,10 +416,10 @@ private void verifyExtraHaplotypeAssignment()
416416
goToAssayRun("first run");
417417
drt = new DataRegionTable("Data", this);
418418
drt.setFilter("AnimalId", "Equals", "ID-5");
419-
verifyColumnDataValues(drt, "MHC-AHaplotype1", "A001");
420-
verifyColumnDataValues(drt, "MHC-AHaplotype2", "A002a");
421-
verifyColumnDataValues(drt, "MHC-BHaplotype1", "B002");
422-
verifyColumnDataValues(drt, "MHC-BHaplotype2", "B002");
419+
verifyColumnDataValues(drt, "mhcAHaplotype1", "A001");
420+
verifyColumnDataValues(drt, "mhcAHaplotype2", "A002a");
421+
verifyColumnDataValues(drt, "mhcBHaplotype1", "B002");
422+
verifyColumnDataValues(drt, "mhcBHaplotype2", "B002");
423423
drt.clearFilter("AnimalId");
424424

425425
// NOTE: this should clean up what it has done in order to make the test more modular...

0 commit comments

Comments
 (0)