trustpub: Show "Trusted Publishing" crate settings for everyone

The feedback on our semi-public testing period has been positive, so let's make this a public testing period now.

Author: Turbo87

app/templates/crate/settings/index.hbs

@@ -54,51 +54,48 @@
   {{/each}}
 </div>
 
-{{! The "Trusted Publishing" section is hidden for now until we make this feature publicly available. }}
-{{#if this.githubConfigs}}
-  <div local-class="header">
-    <h2>Trusted Publishing</h2>
-    <div>
-      <LinkTo @route="docs.trusted-publishing" class="button button--tan button--small" data-test-trusted-publishing-docs-button>
-        Learn more
-      </LinkTo>
-      <LinkTo @route="crate.settings.new-trusted-publisher" class="button button--small" data-test-add-trusted-publisher-button>
-        Add
-      </LinkTo>
-    </div>
+<div local-class="header">
+  <h2>Trusted Publishing</h2>
+  <div>
+    <LinkTo @route="docs.trusted-publishing" class="button button--tan button--small" data-test-trusted-publishing-docs-button>
+      Learn more
+    </LinkTo>
+    <LinkTo @route="crate.settings.new-trusted-publisher" class="button button--small" data-test-add-trusted-publisher-button>
+      Add
+    </LinkTo>
   </div>
+</div>
 
-  <table local-class="trustpub" data-test-trusted-publishing>
-    <thead>
-    <tr>
-      <th>Publisher</th>
-      <th>Details</th>
-      <th></th>
+<table local-class="trustpub" data-test-trusted-publishing>
+  <thead>
+  <tr>
+    <th>Publisher</th>
+    <th>Details</th>
+    <th></th>
+  </tr>
+  </thead>
+  <tbody>
+  {{#each this.githubConfigs as |config|}}
+    <tr data-test-github-config={{config.id}}>
+      <td>GitHub</td>
+      <td local-class="details">
+        <strong>Repository:</strong> {{config.repository_owner}}/{{config.repository_name}}<br>
+        <strong>Workflow:</strong> {{config.workflow_filename}}<br>
+        {{#if config.environment}}
+          <strong>Environment:</strong> {{config.environment}}
+        {{/if}}
+      </td>
+      <td local-class="actions">
+        <button type="button" class="button button--small" data-test-remove-config-button {{on "click" (perform this.removeConfigTask config)}}>Remove</button>
+      </td>
     </tr>
-    </thead>
-    <tbody>
-    {{#each this.githubConfigs as |config|}}
-      <tr data-test-github-config={{config.id}}>
-        <td>GitHub</td>
-        <td local-class="details">
-          <strong>Repository:</strong> {{config.repository_owner}}/{{config.repository_name}}<br>
-          <strong>Workflow:</strong> {{config.workflow_filename}}<br>
-          {{#if config.environment}}
-            <strong>Environment:</strong> {{config.environment}}
-          {{/if}}
-        </td>
-        <td local-class="actions">
-          <button type="button" class="button button--small" data-test-remove-config-button {{on "click" (perform this.removeConfigTask config)}}>Remove</button>
-        </td>
-      </tr>
-    {{else}}
-      <tr data-test-no-config>
-        <td colspan="3">No trusted publishers configured for this crate.</td>
-      </tr>
-    {{/each}}
-    </tbody>
-  </table>
-{{/if}}
+  {{else}}
+    <tr data-test-no-config>
+      <td colspan="3">No trusted publishers configured for this crate.</td>
+    </tr>
+  {{/each}}
+  </tbody>
+</table>
 
 <h2 local-class="header">Danger Zone</h2>
 

e2e/routes/crate/settings.spec.ts

@@ -51,11 +51,9 @@ test.describe('Route | crate.settings', { tag: '@routes' }, () => {
     await expect(page.locator(`[data-test-owner-user="${user.login}"]`)).toBeVisible();
     await expect(page.locator('[data-test-remove-owner-button]')).toBeVisible();
 
-    // Disabled for now, until we make this feature publicly available
-    // await expect(page.locator('[data-test-trusted-publishing]')).toBeVisible();
-    // await expect(page.locator('[data-test-no-config]')).toBeVisible();
-    // await expect(page.locator('[data-test-github-config]')).not.toBeVisible();
-    await expect(page.locator('[data-test-trusted-publishing]')).not.toBeVisible();
+    await expect(page.locator('[data-test-trusted-publishing]')).toBeVisible();
+    await expect(page.locator('[data-test-no-config]')).toBeVisible();
+    await expect(page.locator('[data-test-github-config]')).not.toBeVisible();
 
     await expect(page.locator('[data-test-delete-button]')).toBeVisible();
   });

tests/routes/crate/settings-test.js

@@ -55,11 +55,9 @@ module('Route | crate.settings', hooks => {
     assert.dom(`[data-test-owner-user="${user.login}"]`).exists();
     assert.dom('[data-test-remove-owner-button]').exists();
 
-    // Disabled for now, until we make this feature publicly available
-    // assert.dom('[data-test-trusted-publishing]').exists();
-    // assert.dom('[data-test-no-config]').exists();
-    // assert.dom('[data-test-github-config]').doesNotExist();
-    assert.dom('[data-test-trusted-publishing]').doesNotExist();
+    assert.dom('[data-test-trusted-publishing]').exists();
+    assert.dom('[data-test-no-config]').exists();
+    assert.dom('[data-test-github-config]').doesNotExist();
 
     assert.dom('[data-test-delete-button]').exists();
   });