Skip to content

Commit

Permalink
Update slug in code comments
Browse files Browse the repository at this point in the history
  • Loading branch information
yanokwa committed Jul 6, 2020
1 parent e72526a commit 568b141
Show file tree
Hide file tree
Showing 21 changed files with 28 additions and 29 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
import static org.odk.collect.android.feature.externalintents.ExportedActivitiesUtils.testDirectories;

@Suppress
// Frequent failures: https://github.com/opendatakit/collect/issues/796
// Frequent failures: https://github.com/getodk/collect/issues/796
public class FormChooserListTest {

@Rule
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
import static org.odk.collect.android.feature.externalintents.ExportedActivitiesUtils.testDirectories;

@Suppress
// Frequent failures: https://github.com/opendatakit/collect/issues/796
// Frequent failures: https://github.com/getodk/collect/issues/796
@RunWith(AndroidJUnit4.class)
public class FormEntryActivityTest {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
import static org.odk.collect.android.feature.externalintents.ExportedActivitiesUtils.testDirectories;

@Suppress
// Frequent failures: https://github.com/opendatakit/collect/issues/796
// Frequent failures: https://github.com/getodk/collect/issues/796
@RunWith(AndroidJUnit4.class)
public class InstanceChooserListTest {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
import static org.odk.collect.android.feature.externalintents.ExportedActivitiesUtils.testDirectories;

@Suppress
// Frequent failures: https://github.com/opendatakit/collect/issues/796
// Frequent failures: https://github.com/getodk/collect/issues/796
public class MainMenuActivityTest {

@Rule
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ public void search_withoutFilter_displaysAllChoices() {
}

@Test
// Regression: https://github.com/opendatakit/collect/issues/3132
// Regression: https://github.com/getodk/collect/issues/3132
public void search_withFilter_showsMatchingChoices() {
new FormEntryPage("external-csv-search", rule).assertOnPage()
.swipeToNextQuestion("Produce search")
Expand All @@ -56,7 +56,6 @@ public void search_withFilter_showsMatchingChoices() {
.assertText("Cherimoya")
.assertText("Carrot")
.checkIfTextDoesNotExist("Blueberry")

.swipeToPreviousQuestion()
.inputText("B")
.swipeToNextQuestion("Produce")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ public class FormHierarchyTest {
.around(rule);

@Test
//https://github.com/opendatakit/collect/issues/2871
//https://github.com/getodk/collect/issues/2871
public void allRelevantQuestionsShouldBeVisibleInHierarchyView() {
new MainMenuPage(rule)
.startBlankForm("formHierarchy1")
Expand All @@ -56,7 +56,7 @@ public void allRelevantQuestionsShouldBeVisibleInHierarchyView() {
}

@Test
//https://github.com/opendatakit/collect/issues/2944
//https://github.com/getodk/collect/issues/2944
public void notRelevantRepeatGroupsShouldNotBeVisibleInHierarchy() {
final FormHierarchyPage page = new MainMenuPage(rule)
.startBlankForm("formHierarchy2")
Expand Down Expand Up @@ -102,7 +102,7 @@ public void notRelevantRepeatGroupsShouldNotBeVisibleInHierarchy() {
}

@Test
//https://github.com/opendatakit/collect/issues/2936
//https://github.com/getodk/collect/issues/2936
public void repeatGroupsShouldBeVisibleAsAppropriate() {
FormHierarchyPage page = new MainMenuPage(rule)
.startBlankForm("formHierarchy3")
Expand Down Expand Up @@ -131,7 +131,7 @@ public void repeatGroupsShouldBeVisibleAsAppropriate() {
}

@Test
//https://github.com/opendatakit/collect/issues/2942
//https://github.com/getodk/collect/issues/2942
public void deletingLastGroupShouldNotBreakHierarchy() {
FormHierarchyPage page = new MainMenuPage(rule)
.startBlankForm("formHierarchy3")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ public abstract class SqlLiteHelperTest {
*
* Without this, it appears that the migrations only get partially applied. It's not clear how
* this is possible since calls to onDowngrade and onUpgrade are wrapped in transactions. See
* discussion at https://github.com/opendatakit/collect/pull/3250#issuecomment-516439704
* discussion at https://github.com/getodk/collect/pull/3250#issuecomment-516439704
*/
void ensureMigrationAppliesFully(SQLiteOpenHelper databaseHelper) {
databaseHelper.getReadableDatabase().close();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ public class FormUtilsTest {
.around(new CopyFormRule(BASIC_FORM));

/* Verify that each host string matches only a single root translator, allowing for them to
be defined in any order. See: https://github.com/opendatakit/collect/issues/3334
be defined in any order. See: https://github.com/getodk/collect/issues/3334
*/
@Test
public void sessionRootTranslatorOrderDoesNotMatter() throws Exception {
Expand Down Expand Up @@ -63,7 +63,7 @@ public void sessionRootTranslatorOrderDoesNotMatter() throws Exception {
matching 'file' instead of 'file-csv'. According to the behavior in the test above,
sessionRootTranslatorOrderDoesNotMatter, it is not actually a requirement to have the test
below pass. This simply follows the cautionary remarks in the following issue:
https://github.com/opendatakit/collect/issues/3334
https://github.com/getodk/collect/issues/3334
*/
@Test
public void hostStringsOrderedCorrectly() throws Exception {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ public void searchExpression_ShouldDisplayWhenItContainsOtherAppearanceName() {

//TestCase26
// This form doesn't define an instanceID and also doesn't request encryption so this case
// would catch regressions for https://github.com/opendatakit/collect/issues/3340
// would catch regressions for https://github.com/getodk/collect/issues/3340
new MainMenuPage(rule).startBlankForm("CSV error Form")
.clickOnText("Greg Pommen")
.swipeToNextQuestion()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -895,7 +895,7 @@ protected void onActivityResult(int requestCode, int resultCode, final Intent in
Probably this approach should be used in all cases to get a file from an uri.
The approach which was used before and which is still used in other places
might be faulty because sometimes _data column might be not provided in an uri.
e.g. https://github.com/opendatakit/collect/issues/705
e.g. https://github.com/getodk/collect/issues/705
Let's test it here and then we can use the same code in other places if it works well.
*/
Uri mediaUri = intent.getData();
Expand Down Expand Up @@ -2791,7 +2791,7 @@ private void updateFieldListQuestions(FormIndex lastChangedIndex) {
// identification and removal in the same pass but removal has to be done in a loop that
// starts from the end and itemset-based select choices will only be correctly recomputed
// if accessed from beginning to end because the call on sameAs is what calls
// populateDynamicChoices. See https://github.com/opendatakit/javarosa/issues/436
// populateDynamicChoices. See https://github.com/github.com/getodk//javarosa/issues/436
List<FormEntryPrompt> questionsThatHaveNotChanged = new ArrayList<>();
List<FormIndex> formIndexesToRemove = new ArrayList<>();
for (ImmutableDisplayableQuestion questionBeforeSave : immutableQuestionsBeforeSave) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ public void onCreate(Bundle savedInstanceState) {
setSupportActionBar(toolbar);

FormController formController = Collect.getInstance().getFormController();
// https://github.com/opendatakit/collect/issues/998
// https://github.com/getodk/collect/issues/998
if (formController == null) {
finish();
Timber.w("FormController is null");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@
import static org.odk.collect.android.preferences.MetaKeys.KEY_GOOGLE_BUG_154855417_FIXED;

/**
* The Open Data Kit Collect application.
* The ODK Collect application.
*
* @author carlhartung
*/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -241,7 +241,7 @@ private void addAltitudeAndAccuracyTitles(List<Object> sheetHeaders, List<Object
}

// Ignore rows with all empty answers added by a user and extra repeatable groups added
// by Javarosa https://github.com/opendatakit/javarosa/issues/266
// by Javarosa https://github.com/getodk/javarosa/issues/266
private boolean shouldRowBeInserted(HashMap<String, String> answers) {
for (String answer : answers.values()) {
if (answer != null && !answer.trim().isEmpty()) {
Expand Down Expand Up @@ -531,7 +531,7 @@ private List<Object> prepareListOfValues(List<Object> columnHeaders, List<Object
answer = answers.get(path.toString());
}
}
// https://github.com/opendatakit/collect/issues/931
// https://github.com/getodk/collect/issues/931
list.add(answer.isEmpty() ? " " : answer);
}
return list;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ private List<String> getPackageName(Context context) {
return packagesSupportingCustomTabs;
}

// https://github.com/opendatakit/collect/issues/1221
// https://github.com/getodk/collect/issues/1221
private Uri getNonNullUri(Uri url) {
return url != null ? url : Uri.parse("");
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,7 @@ private static void scaleDownImage(String imagePath, int maxPixels) {

/**
* Sometimes an image might be taken up sideways.
* https://github.com/opendatakit/collect/issues/36
* https://github.com/getodk/collect/issues/36
*/
private static void rotateImageIfNeeded(String imagePath) {
ExifInterface exif = null;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
* initializer that initializes SpacesInUnderlyingValuesWarning with the new renderer
* as in {@link ##forQuestionWidget(QuestionWidget)}
*
* Created as fix for https://github.com/opendatakit/collect/issues/1964
* Created as fix for https://github.com/getodk/collect/issues/1964
*
*/
public class SpacesInUnderlyingValuesWarning {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ public void doImport_createsDataAndMetadataTables() {

/**
* There are multiple features that ingest CSV files so the original file should not be modified.
* https://github.com/opendatakit/collect/issues/3335
* https://github.com/getodk/collect/issues/3335
*/
@Test
public void doImport_doesNotModifyOriginalCsv() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public void when204Response_returnsHeaders() throws Exception {
}

// Ensure we can look up lower-case headers using mixed-case header names.
// https://github.com/opendatakit/collect/issues/3068
// https://github.com/getodk/collect/issues/3068
@Test
public void when204Response_returnsLowerCaseHeaders() throws Exception {
String headerLowerCase = "header-case-test";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@
import static org.mockito.Mockito.when;

@RunWith(RobolectricTestRunner.class)
/** https://github.com/opendatakit/collect/issues/356
/** https://github.com/getodk/collect/issues/356
* Verify that the {@link DateWidget} and {@link DateTimeWidget} widget skips over
* "daylight savings gaps".
* This is needed on the day and time of a daylight savings transition because that date/time
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ public OpenRosaAPIClient provideCollectServerClient(OpenRosaHttpInterface httpIn

/**
* Verifies that a form without media can successfully go through the download process. Regression
* test for https://github.com/opendatakit/collect/issues/3535.
* test for https://github.com/getodk/collect/issues/3535.
*
* The focus of this test is the form parsing behavior triggered by a download and how it
* relates to a media folder that may or may not have been created. The downloading of forms and
Expand Down Expand Up @@ -167,7 +167,7 @@ public void downloadingFormWithMedia_Succeeds() throws Exception {
* Forms with references to external secondary instance need to have the secondary instance
* available at time of form parse.
*
* See https://github.com/opendatakit/collect/issues/3635
* See https://github.com/getodk/collect/issues/3635
*/
@Test
public void downloadingFormWithXmlExternalSecondaryInstance_Succeeds() throws Exception {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ public class StringUtilsTest {
* <p>
* This is a silly test but it is here to guarantee this behaviour,
* since without it the method causes a crash when processing text for
* questions with no plain text label. See opendatakit/opendatakit#1247.
* questions with no plain text label. See getodk/getodk#1247.
*/
@Test
public void textToHtml_BouncesNullInput() {
Expand Down

0 comments on commit 568b141

Please sign in to comment.