Apply alpha conversions to reduce the word *ember*.

- `USE_EXTERNAL_EMBER` -> `USE_EXTERNAL_WEB_APP`
- `PROXY_EMBER_APPS` -> `PROXY_WEB_APPS`
- `EXTERNAL_EMBER_APPS` -> `EXTERNAL_WEB_APPS`
- `EXTERNAL_EMBER_SERVER_TIMEOUT` -> `EXTERNAL_APP_SERVER_TIMEOUT`
- `EMBER_DOMAIN` -> `PRIMARY_WEB_APP_DOMAIN`

- `stream_emberapp` -> `stream_web_app`
- `use_ember_app` -> `use_primary_web_app`
  - Identifiers includes `use_ember_app` as part of name are also
    converted. same rule.
- directory name `ember_osf_web` -> `external_web_app`
  - affected `import` statements are fixed, too.

Author: chiku

.docker-compose.env

@@ -17,6 +17,6 @@ MFR_SERVER_URL=http://localhost:7778
 #SHARE_URL=http://192.168.168.167:8000/
 BROKER_URL=amqp://guest:guest@192.168.168.167:5672/
 REDIS_URL=redis://192.168.168.167:6379/1
-EMBER_DOMAIN=192.168.168.167
+PRIMARY_WEB_APP_DOMAIN=192.168.168.167
 
 #PYTHONUNBUFFERED=0 # This when set to 0 will allow print statements to be visible in the Docker logs

addons/osfstorage/tests/test_views.py

@@ -36,7 +36,7 @@
 
 from osf_tests.factories import ProjectFactory, ApiOAuth2PersonalTokenFactory, PreprintFactory
 from website.files.utils import attach_versions
-from website.settings import EXTERNAL_EMBER_APPS
+from website.settings import EXTERNAL_WEB_APPS
 from api_tests.draft_nodes.views.test_draft_node_files_lists import prepare_mock_wb_response
 
 
@@ -1419,7 +1419,7 @@ def test_file_view_updates_history(self):
             file = GithubFile.objects.get(_path='/testpath', provider='github')
             assert file.history
 
-    @mock.patch('website.views.stream_emberapp')
+    @mock.patch('website.views.stream_web_app')
     def test_file_views(self, mock_ember):
         with override_flag(features.EMBER_FILE_PROJECT_DETAIL, active=True):
             file = create_test_file(target=self.node, user=self.user)
@@ -1433,8 +1433,8 @@ def test_file_views(self, mock_ember):
             assert mock_ember.called
             args, kwargs = mock_ember.call_args
 
-            assert args[0] == EXTERNAL_EMBER_APPS['ember_osf_web']['server']
-            assert args[1] == EXTERNAL_EMBER_APPS['ember_osf_web']['path'].rstrip('/')
+            assert args[0] == EXTERNAL_WEB_APPS['ember_osf_web']['server']
+            assert args[1] == EXTERNAL_WEB_APPS['ember_osf_web']['path'].rstrip('/')
 
     def test_download_file(self):
         file = create_test_file(target=self.node, user=self.user)

addons/wiki/views.py

@@ -17,7 +17,7 @@
 from website.profile.utils import get_profile_image_url
 from website.project.views.node import _view_project
 from website.project.model import has_anonymous_link
-from website.ember_osf_web.decorators import ember_flag_is_active
+from website.external_web_app.decorators import ember_flag_is_active
 from website.project.decorators import (
     must_be_contributor_or_public,
     must_have_addon, must_not_be_registration,

framework/auth/views.py

@@ -35,7 +35,7 @@
 from osf.models.tag import Tag
 from osf.utils.requests import check_select_for_update
 from website.util.metrics import CampaignClaimedTags, CampaignSourceTags
-from website.ember_osf_web.decorators import ember_flag_is_active
+from website.external_web_app.decorators import ember_flag_is_active
 from osf import features
 
 

osf_tests/test_guid.py

@@ -269,8 +269,8 @@ def test_resolve_guid_download_file(self):
         assert res.status_code == 302
         assert f'{WATERBUTLER_URL}/v1/resources/{unpub_pp._id}/providers/{unpub_pp.primary_file.provider}{unpub_pp.primary_file.path}?version=1&action=download&direct' in res.location
 
-    @mock.patch('website.settings.USE_EXTERNAL_EMBER', True)
-    @mock.patch('website.settings.EXTERNAL_EMBER_APPS', {
+    @mock.patch('website.settings.USE_EXTERNAL_WEB_APP', True)
+    @mock.patch('website.settings.EXTERNAL_WEB_APPS', {
         'preprints': {
             'server': 'http://localhost:4200',
             'path': '/preprints/'
@@ -289,8 +289,8 @@ def test_resolve_guid_download_file_from_emberapp_preprints(self):
         assert res.status_code == 302
         assert f'{WATERBUTLER_URL}/v1/resources/{pp._id}/providers/{pp.primary_file.provider}{pp.primary_file.path}?action=download&direct&version=1' in res.location
 
-    @mock.patch('website.settings.USE_EXTERNAL_EMBER', True)
-    @mock.patch('website.settings.EXTERNAL_EMBER_APPS', {
+    @mock.patch('website.settings.USE_EXTERNAL_WEB_APP', True)
+    @mock.patch('website.settings.EXTERNAL_WEB_APPS', {
         'preprints': {
             'server': 'http://localhost:4200',
             'path': '/preprints/'

tests/test_addons.py

@@ -46,7 +46,7 @@
 from framework import sentry
 from api.base.settings.defaults import API_BASE
 from tests.json_api_test_app import JSONAPITestApp
-from website.settings import EXTERNAL_EMBER_APPS
+from website.settings import EXTERNAL_WEB_APPS
 from waffle.testutils import override_flag
 from django.conf import settings as django_conf_settings
 
@@ -1360,7 +1360,7 @@ def test_action_download_redirects_to_download_with_version(self):
         # Note: version is added but us but all other url params are added as well
         assert_urls_equal(location.url, file_node.generate_waterbutler_url(action='download', direct=None, revision=1, version=''))
 
-    @mock.patch('website.views.stream_emberapp')
+    @mock.patch('website.views.stream_web_app')
     @pytest.mark.enable_bookmark_creation
     def test_action_view_calls_view_file(self, mock_ember):
         self.user.reload()
@@ -1374,10 +1374,10 @@ def test_action_view_calls_view_file(self, mock_ember):
 
         args, kwargs = mock_ember.call_args
         assert kwargs == {}
-        assert args[0] == EXTERNAL_EMBER_APPS['ember_osf_web']['server']
-        assert args[1] == EXTERNAL_EMBER_APPS['ember_osf_web']['path'].rstrip('/')
+        assert args[0] == EXTERNAL_WEB_APPS['ember_osf_web']['server']
+        assert args[1] == EXTERNAL_WEB_APPS['ember_osf_web']['path'].rstrip('/')
 
-    @mock.patch('website.views.stream_emberapp')
+    @mock.patch('website.views.stream_web_app')
     @pytest.mark.enable_bookmark_creation
     def test_no_action_calls_view_file(self, mock_ember):
         self.user.reload()
@@ -1391,8 +1391,8 @@ def test_no_action_calls_view_file(self, mock_ember):
 
         args, kwargs = mock_ember.call_args
         assert kwargs == {}
-        assert args[0] == EXTERNAL_EMBER_APPS['ember_osf_web']['server']
-        assert args[1] == EXTERNAL_EMBER_APPS['ember_osf_web']['path'].rstrip('/')
+        assert args[0] == EXTERNAL_WEB_APPS['ember_osf_web']['server']
+        assert args[1] == EXTERNAL_WEB_APPS['ember_osf_web']['path'].rstrip('/')
 
     def test_download_create_guid(self):
         file_node = self.get_test_file()
@@ -1462,7 +1462,7 @@ def test_nonstorage_addons_raise(self):
         )
         assert resp.status_code == 400
 
-    @mock.patch('website.views.stream_emberapp')
+    @mock.patch('website.views.stream_web_app')
     def test_head_returns_url_and_redriect(self, mock_ember):
         file_node = self.get_test_file()
         guid = file_node.get_guid(create=True)
@@ -1473,8 +1473,8 @@ def test_head_returns_url_and_redriect(self, mock_ember):
 
         args, kwargs = mock_ember.call_args
         assert kwargs == {}
-        assert args[0] == EXTERNAL_EMBER_APPS['ember_osf_web']['server']
-        assert args[1] == EXTERNAL_EMBER_APPS['ember_osf_web']['path'].rstrip('/')
+        assert args[0] == EXTERNAL_WEB_APPS['ember_osf_web']['server']
+        assert args[1] == EXTERNAL_WEB_APPS['ember_osf_web']['path'].rstrip('/')
 
 
     def test_head_returns_url_with_version_and_redirect(self):

tests/test_ember_osf_web.py

@@ -2,7 +2,7 @@
 from flask import request
 
 from tests.base import OsfTestCase
-from website.ember_osf_web.decorators import ember_flag_is_active
+from website.external_web_app.decorators import ember_flag_is_active
 from osf_tests.factories import FlagFactory, UserFactory
 
 from django.contrib.auth.models import Group
@@ -16,24 +16,26 @@ def setUp(self):
         FlagFactory(name='inactive_flag', everyone=False).save()
         self.mock_func = lambda: 'test value'
 
-    @mock.patch('website.ember_osf_web.decorators.use_ember_app')
-    def test_use_ember_app(self, mock_use_ember_app):
+    # TODO: This test case is not Ember App specific.
+    @mock.patch('website.external_web_app.decorators.use_primary_web_app')
+    def test_use_primary_web_app(self, mock_use_primary_web_app):
         ember_flag_is_active('active_flag')(self.mock_func)()
 
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()
 
-    @mock.patch('website.ember_osf_web.decorators.use_ember_app')
-    def test_dont_use_ember_app(self, mock_use_ember_app):
+    # TODO: This test case is not Ember App specific.
+    @mock.patch('website.external_web_app.decorators.use_primary_web_app')
+    def test_dont_use_primary_web_app(self, mock_use_primary_web_app):
         # mock over external module 'waflle.flag_is_active` not ours
 
         ember_flag_is_active('inactive_flag')(self.mock_func)()
 
-        assert not mock_use_ember_app.called
+        assert not mock_use_primary_web_app.called
 
     @mock.patch('api.waffle.utils._get_current_user')
-    @mock.patch('website.ember_osf_web.decorators.flag_is_active')
-    @mock.patch('website.ember_osf_web.decorators.use_ember_app')
-    def test_ember_flag_is_active_authenticated_user(self, mock_use_ember_app, mock_flag_is_active, mock__get_current_user):
+    @mock.patch('website.external_web_app.decorators.flag_is_active')
+    @mock.patch('website.external_web_app.decorators.use_primary_web_app')
+    def test_ember_flag_is_active_authenticated_user(self, mock_use_primary_web_app, mock_flag_is_active, mock__get_current_user):
         # mock over external module 'waflle.flag_is_active` not ours
 
         user = UserFactory()
@@ -42,12 +44,12 @@ def test_ember_flag_is_active_authenticated_user(self, mock_use_ember_app, mock_
         ember_flag_is_active('active_flag')(self.mock_func)()
 
         mock_flag_is_active.assert_called_with(request, 'active_flag')
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()
 
     @mock.patch('api.waffle.utils._get_current_user', return_value=None)
-    @mock.patch('website.ember_osf_web.decorators.flag_is_active')
-    @mock.patch('website.ember_osf_web.decorators.use_ember_app')
-    def test_ember_flag_is_active_unauthenticated_user(self, mock_use_ember_app, mock_flag_is_active, mock__get_current_user):
+    @mock.patch('website.external_web_app.decorators.flag_is_active')
+    @mock.patch('website.external_web_app.decorators.use_primary_web_app')
+    def test_ember_flag_is_active_unauthenticated_user(self, mock_use_primary_web_app, mock_flag_is_active, mock__get_current_user):
         # mock over external module 'waflle.flag_is_active` not ours
 
         ember_flag_is_active('active_flag')(self.mock_func)()
@@ -56,4 +58,4 @@ def test_ember_flag_is_active_unauthenticated_user(self, mock_use_ember_app, moc
         self.flag.groups.add(group)
 
         mock_flag_is_active.assert_called_with(request, 'active_flag')
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()

tests/test_misc_views.py

@@ -775,27 +775,27 @@ def test_find_unread_includes_edited_comments(self):
         n_unread = Comment.find_n_unread(user=user, node=project, page='node')
         assert n_unread == 1
 
-@mock.patch('website.views.PROXY_EMBER_APPS', False)
+@mock.patch('website.views.PROXY_WEB_APPS', False)
 class TestResolveGuid(OsfTestCase):
     def setUp(self):
         super().setUp()
 
-    @mock.patch('website.views.use_ember_app')
-    def test_preprint_provider_without_domain(self, mock_use_ember_app):
+    @mock.patch('website.views.use_primary_web_app')
+    def test_preprint_provider_without_domain(self, mock_use_primary_web_app):
         provider = PreprintProviderFactory(domain='')
         preprint = PreprintFactory(provider=provider)
         url = web_url_for('resolve_guid', _guid=True, guid=preprint._id)
         res = self.app.get(url)
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()
 
-    @mock.patch('website.views.use_ember_app')
-    def test_preprint_provider_with_domain_without_redirect(self, mock_use_ember_app):
+    @mock.patch('website.views.use_primary_web_app')
+    def test_preprint_provider_with_domain_without_redirect(self, mock_use_primary_web_app):
         domain = 'https://test.com/'
         provider = PreprintProviderFactory(_id='test', domain=domain, domain_redirect_enabled=False)
         preprint = PreprintFactory(provider=provider)
         url = web_url_for('resolve_guid', _guid=True, guid=preprint._id)
         res = self.app.get(url)
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()
 
     def test_preprint_provider_with_domain_with_redirect(self):
         domain = 'https://test.com/'
@@ -809,10 +809,10 @@ def test_preprint_provider_with_domain_with_redirect(self):
         assert res.headers['location'] == f'{domain}{preprint._id}/'
         assert res.request.path == f'/{preprint._id}/'
 
-    @mock.patch('website.views.use_ember_app')
-    def test_preprint_provider_with_osf_domain(self, mock_use_ember_app):
+    @mock.patch('website.views.use_primary_web_app')
+    def test_preprint_provider_with_osf_domain(self, mock_use_primary_web_app):
         provider = PreprintProviderFactory(_id='osf', domain='https://osf.io/')
         preprint = PreprintFactory(provider=provider)
         url = web_url_for('resolve_guid', _guid=True, guid=preprint._id)
         res = self.app.get(url)
-        mock_use_ember_app.assert_called_with()
+        mock_use_primary_web_app.assert_called_with()

website/conferences/views.py

@@ -15,7 +15,7 @@
 from website import settings
 from website.conferences import utils
 from website.conferences.message import ConferenceMessage, ConferenceError
-from website.ember_osf_web.decorators import ember_flag_is_active
+from website.external_web_app.decorators import ember_flag_is_active
 from website.mails import CONFERENCE_SUBMITTED, CONFERENCE_INACTIVE, CONFERENCE_FAILED, CONFERENCE_DEPRECATION
 from website.mails import send_mail
 from website.util import web_url_for