Skip to content

Comments

Spinner, toaster notes, date & time picker factories unit tests updates#478

Open
dubdabasoduba wants to merge 15 commits intomasterfrom
feature/testing_tuesday
Open

Spinner, toaster notes, date & time picker factories unit tests updates#478
dubdabasoduba wants to merge 15 commits intomasterfrom
feature/testing_tuesday

Conversation

@dubdabasoduba
Copy link
Member

No description provided.

@dubdabasoduba dubdabasoduba changed the title Date & Time picker factories unit tests updates Spinner, date & time picker factories unit tests updates Sep 29, 2020
@dubdabasoduba dubdabasoduba changed the title Spinner, date & time picker factories unit tests updates Spinner, toaster notes, date & time picker factories unit tests updates Oct 27, 2020
@dubdabasoduba dubdabasoduba changed the base branch from master to optimize-native-forms November 3, 2020 07:37
import java.util.List;
import java.util.Set;

public class ToasterNotesFactoryTest extends BaseTest {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extend FactoryTest instead, remove the JsonFormActivity declaration and assignment call super on setUp

import java.util.Locale;
import java.util.Set;

public class TimePickerFactoryTest extends BaseTest {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extend FactoryTest instead, remove the JsonFormActivity declaration and assignment call super on setUp

/**
* Created by Vincent Karuri on 03/04/2020
*/
public class SpinnerFactoryTest extends BaseTest {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extend FactoryTest instead, remove the JsonFormActivity declaration and assignment call super on setUp

…ient-native-form into feature/testing_tuesday
Base automatically changed from optimize-native-forms to master January 19, 2021 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants