Revert #59723 (login with SAMS-dev) (#60535)

Author: chrsmith

client/web/src/auth/SignInPage.test.tsx

@@ -143,35 +143,6 @@ describe('SignInPage', () => {
         expect(rendered.asFragment()).toMatchSnapshot()
     })
 
-    describe('with Sourcegraph accounts (dev) auth provider', () => {
-        const samsProviderName = 'Sourcegraph Accounts (dev) [Testing Only]'
-        const withSourcegraphAccountsDev: SourcegraphContext['authProviders'] = [
-            ...authProviders,
-            {
-                displayName: samsProviderName,
-                isBuiltin: false,
-                serviceType: 'openidconnect',
-                authenticationURL: 'https://accounts.sgdev.org/.auth/openidconnect/',
-                serviceID: 'https://accounts.sgdev.org',
-                clientID: 'sams-dev_cid_xxxx',
-            },
-        ]
-
-        it('renders page with 2 providers', () => {
-            const rendered = render('/sign-in', { authProviders: withSourcegraphAccountsDev })
-            expect(
-                within(rendered.baseElement).queryByText(txt => txt.includes(samsProviderName))
-            ).not.toBeInTheDocument()
-            expect(rendered.asFragment()).toMatchSnapshot()
-        })
-
-        it('renders page with 3 providers (url-param present)', () => {
-            const rendered = render('/sign-in?sourcegraph-accounts-dev', { authProviders: withSourcegraphAccountsDev })
-            expect(within(rendered.baseElement).queryByText(txt => txt.includes(samsProviderName))).toBeInTheDocument()
-            expect(rendered.asFragment()).toMatchSnapshot()
-        })
-    })
-
     describe('with Sourcegraph operator auth provider', () => {
         const withSourcegraphOperator: SourcegraphContext['authProviders'] = [
             ...authProviders,

client/web/src/auth/SignInPage.tsx

@@ -60,21 +60,8 @@ export const SignInPage: React.FunctionComponent<React.PropsWithChildren<SignInP
     )
 
     const shouldShowProvider = function (provider: AuthProvider): boolean {
-        const isSourcegraphAccountsDev = (provider: AuthProvider): boolean => {
-            if (provider.serviceType !== 'openidconnect') {
-                return false
-            }
-            if (!provider.displayName.includes('Sourcegraph Accounts (dev)')) {
-                return false
-            }
-            return true
-        }
-
         // Hide the Sourcegraph Operator authentication provider by default because it is
         // not useful to customer users and may even cause confusion.
-        if (isSourcegraphAccountsDev(provider)) {
-            return searchParams.has('sourcegraph-accounts-dev')
-        }
         if (provider.serviceType === 'sourcegraph-operator') {
             return searchParams.has('sourcegraph-operator')
         }