Skip to content

Commit 5b8a915

Browse files
authored
Issue 52461: Luminex Levey-Jennings report 4PL curve fit comparison report not working for assay run selected from another folder (#851)
- add LABKEY.Query.containerFilter.allFolders to selectRows request in viewCurvesClicked()
1 parent 56487dc commit 5b8a915

File tree

3 files changed

+16
-4
lines changed

3 files changed

+16
-4
lines changed

luminex/test/src/org/labkey/test/tests/luminex/LuminexGuideSetTest.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,10 @@ private void verifyGuideSetThresholds(Map<String, Integer> guideSetIds, String[]
384384
@LogMethod
385385
private void verifyLeveyJenningsPlots()
386386
{
387+
// Issue 52461: view the LJ report and curve fit comparison plots from subfolder
388+
goToProjectFolder(getProjectName(), TEST_ASSAY_SUBFOLDER);
389+
clickAndWait(Locator.linkWithText(TEST_ASSAY_LUM));
390+
387391
_guideSetHelper.goToLeveyJenningsGraphPage(TEST_ASSAY_LUM, "Standard1");
388392
_guideSetHelper.setUpLeveyJenningsGraphParams("GS Analyte B");
389393

@@ -434,6 +438,8 @@ private void verifyLeveyJenningsPlots()
434438
selectCurveComparisonPlotOption("curvecomparison-legend-combo", "Notebook No.");
435439
Locator.extButton("Close").findElement(curveComparisonWindow).click();
436440
_extHelper.waitForExt3MaskToDisappear(WAIT_FOR_JAVASCRIPT);
441+
442+
goToProjectHome();
437443
}
438444

439445
private void selectCurveComparisonPlotOption(String comboName, String value)

luminex/test/src/org/labkey/test/tests/luminex/LuminexTest.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,8 @@
6464
@BaseWebDriverTest.ClassTimeout(minutes = 40)
6565
public abstract class LuminexTest extends BaseWebDriverTest
6666
{
67-
protected final static String TEST_ASSAY_PRJ_LUMINEX = "LuminexTest Project"; //project for luminex test
67+
protected final static String TEST_ASSAY_PRJ_LUMINEX = "LuminexTest Project";
68+
protected final static String TEST_ASSAY_SUBFOLDER = "Subfolder";//project for luminex test
6869

6970
public static final String TEST_ASSAY_LUM = "&TestAssayLuminex></% 1";// put back TRICKY_CHARACTERS_NO_QUOTES when issue 20061 is resolved
7071
protected static final String TEST_ASSAY_LUM_DESC = "Description for Luminex assay";
@@ -181,6 +182,10 @@ private void doInit() throws Exception
181182
goToProjectHome();
182183
}
183184

185+
// create a subfolder to the project
186+
_containerHelper.createSubfolder(getProjectName(), TEST_ASSAY_SUBFOLDER, "Assay");
187+
goToProjectHome();
188+
184189
PortalHelper portalHelper = new PortalHelper(this);
185190
//add the Assay List web part so we can create a new luminex assay
186191
portalHelper.addWebPart("Assay List");
@@ -575,7 +580,7 @@ protected void doCleanup(boolean afterTest) throws TestTimeoutException
575580
//helper function to go to test assay home from anywhere the project link is visible
576581
public void goToTestAssayHome(String assayName)
577582
{
578-
if (!isTextPresent(assayName + " Runs"))
583+
if (!getDriver().getTitle().startsWith(assayName + " Runs"))
579584
{
580585
goToProjectHome();
581586
clickAndWait(Locator.linkWithText(assayName));

luminex/webapp/luminex/LeveyJenningsTrackingDataPanel.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,6 +257,7 @@ LABKEY.LeveyJenningsTrackingDataPanel = Ext.extend(Ext.Component, {
257257
queryName: this.controlType === 'SinglePoint' ? 'AnalyteSinglePointControl' : 'AnalyteTitration',
258258
columns: 'Analyte/Data/Run/RowId',
259259
filterArray: [LABKEY.Filter.create('Analyte', analyteIds, LABKEY.Filter.Types.IN)],
260+
containerFilter: LABKEY.Query.containerFilter.allFolders,
260261
success: function(data) {
261262
var runIds = [];
262263
for (var i = 0; i < data.rows.length; i++) {
@@ -457,11 +458,11 @@ LABKEY.LeveyJenningsTrackingDataPanel = Ext.extend(Ext.Component, {
457458
if (Ext.getDom(divId)) {
458459
var html = Ext.getDom(divId).innerHTML;
459460
html = html.replace(/&amp;/g, "&");
460-
var pdfHref = html.substring(html.indexOf('href="') + 6, html.indexOf('&attachment=true'));
461+
var pdfHref = html.substring(html.indexOf('href="') + 6, html.indexOf('Curve Comparison Plot.pdf') - 2);
461462
if (pdfHref.indexOf("deleteFile") == -1) {
462463
pdfHref = pdfHref + "&deleteFile=false";
463464
}
464-
window.location = pdfHref + "&attachment=true";
465+
window.location = pdfHref;
465466
}
466467
}
467468
},

0 commit comments

Comments
 (0)