diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml new file mode 100644 index 0000000..5a0dd5c --- /dev/null +++ b/.github/workflows/ci.yml @@ -0,0 +1,15 @@ +name: CI + +on: + push: + branches: [ main ] + pull_request: + branches: [ main ] + workflow_dispatch: + +jobs: + islandora-module-ci: + uses: digitalutsc/reusable_workflows/.github/workflows/islandora-module-ci.yml@main + with: + module_name: ableplayer_extend + install_chromedriver: false diff --git a/ableplayer_extend.libraries.yml b/ableplayer_extend.libraries.yml index 1f7a121..e3e87f6 100755 --- a/ableplayer_extend.libraries.yml +++ b/ableplayer_extend.libraries.yml @@ -7,4 +7,4 @@ ableplayer_extend.transcript_position: dependencies: - core/drupal - core/jquery - - core/once \ No newline at end of file + - core/once diff --git a/ableplayer_extend.module b/ableplayer_extend.module index 854c475..ddcdcd1 100755 --- a/ableplayer_extend.module +++ b/ableplayer_extend.module @@ -26,7 +26,8 @@ function ableplayer_extend_help($route_name, RouteMatchInterface $route_match) { /** * Implements hook_field_formatter_third_party_settings_form(). - * Provides additional two config options for Ableplayer + * + * Provides additional two config options for Ableplayer. */ function ableplayer_extend_field_formatter_third_party_settings_form($plugin, $field_definition, $view_mode, $form, $form_state) { @@ -63,7 +64,8 @@ function ableplayer_extend_field_formatter_third_party_settings_form($plugin, $f /** * Implements hook_preprocess_field(). - * Setups the javascript to enable Ableplayer related functions + * + * Setups the javascript to enable Ableplayer related functions. */ function ableplayer_extend_preprocess_field(&$variables) { @@ -83,7 +85,7 @@ function ableplayer_extend_preprocess_field(&$variables) { if (isset($field_display['third_party_settings']['ableplayer_extend'])) { $display_ableplayer_transcript_audio = $field_display['third_party_settings']['ableplayer_extend']['display_ableplayer_transcript_audio']; $show_timestamp_audio = $field_display['third_party_settings']['ableplayer_extend']['show_timestamp_audio']; - + $variables['#attached']['drupalSettings']['display_ableplayer_transcript'] = $display_ableplayer_transcript_audio; $variables['#attached']['drupalSettings']['show_timestamp'] = $show_timestamp_audio; } @@ -102,10 +104,10 @@ function ableplayer_extend_preprocess_field(&$variables) { // Make the setting available in the field template. if (isset($field_display['third_party_settings']['ableplayer_extend'])) { - $display_ableplayer_transcript_video = $field_display['third_party_settings']['ableplayer_extend']['display_ableplayer_transcript_video']; - $show_timestamp_video = $field_display['third_party_settings']['ableplayer_extend']['show_timestamp_video']; - $variables['#attached']['drupalSettings']['display_ableplayer_transcript'] = $display_ableplayer_transcript_video; - $variables['#attached']['drupalSettings']['show_timestamp'] = $show_timestamp_video; + $display_ableplayer_transcript_video = $field_display['third_party_settings']['ableplayer_extend']['display_ableplayer_transcript_video']; + $show_timestamp_video = $field_display['third_party_settings']['ableplayer_extend']['show_timestamp_video']; + $variables['#attached']['drupalSettings']['display_ableplayer_transcript'] = $display_ableplayer_transcript_video; + $variables['#attached']['drupalSettings']['show_timestamp'] = $show_timestamp_video; } $variables['#attached']['library'][] = 'ableplayer_extend/ableplayer_extend.transcript_position'; } diff --git a/composer.json b/composer.json index 0c322d3..63d1ff3 100755 --- a/composer.json +++ b/composer.json @@ -9,6 +9,7 @@ "support": { "issues": "https://github.com/digitalutsc/ableplayer_extend/issues", "source": "https://github.com/digitalutsc/ableplayer_extend" - } + }, + "require": {} } diff --git a/css/ableplayer_transcript.css b/css/ableplayer_transcript.css index 8130b63..902e364 100644 --- a/css/ableplayer_transcript.css +++ b/css/ableplayer_transcript.css @@ -22,4 +22,3 @@ .able-status-bar { height: 2em !important; } - diff --git a/src/Functional/LoadTest.php b/src/Functional/LoadTest.php index 80d0738..1428613 100755 --- a/src/Functional/LoadTest.php +++ b/src/Functional/LoadTest.php @@ -4,14 +4,21 @@ use Drupal\Core\Url; use Drupal\Tests\BrowserTestBase; +use PHPUnit\Framework\Attributes\RunTestsInSeparateProcesses; /** * Simple test to ensure that main page loads with module enabled. * * @group ableplayer_extend */ +#[RunTestsInSeparateProcesses] class LoadTest extends BrowserTestBase { + /** + * {@inheritdoc} + */ + protected $defaultTheme = 'stark'; + /** * Modules to enable. * @@ -40,7 +47,7 @@ protected function setUp(): void { */ public function testLoad() { $this->drupalGet(Url::fromRoute('')); - $this->assertSession()->responseContains('200'); + $this->assertSession()->statusCodeEquals(200); } }