Fix #3294: Create Developer options menu item in Navigation Drawer

[yashraj-01]

Explanation

Fixes #3294: Create Developer options menu item in Navigation Drawer. The feature is backed by tests in NavigationDrawerActivityProdTest and NavigationDrawerActivityDevTest.

Mock link: https://xd.adobe.com/view/e8aa4198-3940-47f9-514a-f41cc54457f6-9e9b/screen/be13be5c-d0c5-4430-bda2-670d48a36b53/

Screenshots

• GIF showing the Developer Options menu item
  

• Espresso Test result for NavigationDrawerActivityProdTest
  

• Robolectric Test result for NavigationDrawerActivityProdTest
  

• Espresso Test result for NavigationDrawerActivityDebugTest
  

• Robolectric Test result for NavigationDrawerActivityDebugTest
  

Checklist

• The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
• The PR explanation includes the words "Fixes #bugnum: ..." (or "Fixes part of #bugnum" if the PR only partially fixes an issue).
• The PR follows the style guide.
• The PR does not contain any unnecessary auto-generated code from Android Studio.
• The PR is made from a branch that's not called "develop".
• The PR is made from a branch that is up-to-date with "develop".
• The PR's branch is based on "develop" and not on any other branch.
• The PR is assigned to an appropriate reviewer in both the Assignees and the Reviewers sections.

[yashraj-01]

There are changes in a lot of files but most of them have same changes which is the addition of DeveloperOptionsStarterModule and DeveloperOptionsModule in the TestApplicationComponent in the test files. Only the following files contain the actual relevant changes:

• AndroidManifest.xml
• DeveloperOptionsActivity.kt
• DeveloperOptionsActivityPresenter.kt
• DeveloperOptionsFragment.kt
• DeveloperOptionsFragmentPresenter.kt
• ExitProfileDialogFragment.kt
• ExitProfileDialogInterface.kt
• NavigationDrawerFooterViewModel.kt
• NavigationDrawerFragment.kt
• NavigationDrawerFragmentPresenter.kt
• FragmentComponent.kt
• developer_options_activity.xml
• developer_options_fragment.xml
• drawer_fragment.xml
• colors.xml
• strings.xml
• DeveloperOptionsActivityTest.kt
• NavigationDrawerActivityProdTest.kt
• NavigationDrawerActivityDebugTest.kt
• arguments.proto

[Sarthak2601]

Good work @yashraj-01. Suggested some changes.
Also, some test cases seem to be failing -- PTAL.

[yashraj-01]

@vinitamurthi I have added the TODOs and filed the issue (#3382). PTAL.

[yashraj-01]

Does this actually require my input @yashraj-01 / @Sarthak2601? Removing myself for now, but if my review is needed please reassign me with specifics on what you'd like me to review.

I assigned you for clarification on Optional. But it is resolved for now. I have filed an issue for that. Apologies for the inconvenience.

[rt4914]

Thanks @yashraj-01

[rt4914]

@yashraj-01 @Sarthak2601 Merge this once all tests pass.

[oppiabot]

Assigning @vinitamurthi for code owner reviews. Thanks!

[Sarthak2601]

LGTM, thanks!

[Sarthak2601]

@vinitamurthi PTAL for code owners, thanks!

[vinitamurthi]

I mainly checked if the TODOs have been added and looks like they have so LGTM!

[vinitamurthi]

Ah sorry I see there was a proto as well. Proto also LGTM. Merging this now!