Restore tests that were skipped as part of the redesign

Author: kategengler

app/components/table-of-contents.hbs

@@ -1,5 +1,5 @@
 <label class='toc-private-toggle'>
-  <input type='checkbox' checked={{@showPrivateClasses}} onchange={{@togglePrivateClasses}} class='private-deprecated-toggle' />
+  <input type='checkbox' checked={{@showPrivateClasses}} onchange={{@togglePrivateClasses}} class='private-deprecated-toggle' data-test-private-deprecated-toggle />
   Show Private / Deprecated packages
 </label>
 <ul class='table-of-contents '>

tests/acceptance/function-test.js

@@ -1,20 +1,16 @@
 /* eslint-disable qunit/no-assert-equal */
 import { module, test } from 'qunit';
 import { setupApplicationTest } from 'ember-qunit';
-import { visit, find } from '@ember/test-helpers';
+import { visit } from '@ember/test-helpers';
 
 module('Acceptance | Function', function (hooks) {
   setupApplicationTest(hooks);
 
-  test.skip('shows function when loaded from url', async function (assert) {
+  test('shows function when loaded from url', async function (assert) {
     await visit('ember/2.18/functions/@ember%2Fapplication/getOwner');
 
     assert.dom('.method').exists({ count: 1 }, 'Single function per page');
-    assert.equal(
-      find('.method .method-name').innerText,
-      'getOwner',
-      'Correct function is shown',
-    );
+    assert.dom('.method .method-name').hasText('getOwner');
   });
 
   test('shows function when loaded from url in a more modern version', async function (assert) {

tests/acceptance/module-test.js

@@ -1,12 +1,12 @@
 /* eslint-disable qunit/no-assert-equal */
-import { module, test, skip } from 'qunit';
+import { module, test } from 'qunit';
 import { setupApplicationTest } from 'ember-qunit';
 import { visit, click, findAll } from '@ember/test-helpers';
 
 module('Acceptance | Module', function (hooks) {
   setupApplicationTest(hooks);
 
-  skip('lists all public/private classes and namespaces on the module page', async function (assert) {
+  test('lists all public/private classes and namespaces on the module page', async function (assert) {
     await visit('ember/1.0/modules/ember-handlebars');
 
     const store = this.owner.lookup('service:store');
@@ -28,7 +28,7 @@ module('Acceptance | Module', function (hooks) {
       numberPublicClasses + numberNameSpaces,
     );
 
-    await click('.sidebar .private-deprecated-toggle');
+    await click('[data-test-private-deprecated-toggle]');
     assert.equal(
       findAll('.spec-property-list li').length,
       numberPublicClasses + numberNameSpaces + numberPrivateClasses,

tests/acceptance/switch-versions-test.js

@@ -165,7 +165,7 @@ module('Acceptance | version navigation', function (hooks) {
     );
   });
 
-  test.skip('switching versions works if class name includes slashes', async function (assert) {
+  test('switching versions works if class name includes slashes', async function (assert) {
     await visit('/ember/3.4/classes/@ember%2Fobject%2Fcomputed');
     assert.equal(
       currentURL(),
@@ -197,7 +197,7 @@ module('Acceptance | version navigation', function (hooks) {
     );
   });
 
-  test.skip(`switching versions works if we've previously switched for a different class`, async function (assert) {
+  test(`switching versions works if we've previously switched for a different class`, async function (assert) {
     await visit('/ember/3.4/classes/@ember%2Fobject%2Fcomputed');
     assert.equal(
       currentURL(),