Skip to content

Name the data update logging test's key so it is not an old variable name #111

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.

Sign up for GitHub

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

Merged
jwrosewell merged 1 commit into from
Nov 12, 2026
Merged

Name the data update logging test's key so it is not an old variable name #111

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions ...gines/src/test/java/fiftyone/pipeline/engines/services/DataUpdateServiceLoggingTests.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -70,10 +70,10 @@ public class DataUpdateServiceLoggingTests {
// 13 September 2020, 12:26:40 UTC.
private static final long FILE_MODIFIED_MILLIS = 1600000000000L;
// A license key of the length and characters the Distributor issues.
private static final String LICENSE_KEY = "QF8ZR2KD7NMB4XWT9HJC3LPA";
private static final String FULL_KEY = "QF8ZR2KD7NMB4XWT9HJC3LPA";
private static final String MASKED_KEY = "********************3LPA";
private static final String KEYED_URL = UPDATE_URL + "?LicenseKeys=" +
LICENSE_KEY + "&Download=True&Type=HashV41";
FULL_KEY + "&Download=True&Type=HashV41";
private static final String MASKED_URL = UPDATE_URL + "?LicenseKeys=" +
MASKED_KEY + "&Download=True&Type=HashV41";

Expand Down Expand Up @@ -430,7 +430,7 @@ private AspectEngineDataFileDefault newKeyedDataFile(boolean inMemory) {
AspectEngineDataFileDefault file = newDataFile(inMemory, false, false);
DataFileConfiguration config = file.getConfiguration();
config.setDataUpdateLicenseKeys(
Collections.singletonList(LICENSE_KEY));
Collections.singletonList(FULL_KEY));
config.setUrlFormatter(dataFile -> KEYED_URL);
return file;
}
Expand All @@ -449,7 +449,7 @@ private void assertKeyOnlyMasked() {
}
assertFalse(
"The license key is in full in: " + text,
text.contains(LICENSE_KEY));
text.contains(FULL_KEY));
}
}

Expand Down
Loading