Merge pull request #1205 from equalizedigital/steve/no-issue/youtube-video-keyword

updated: video selector to exclude script tags and add test case for YouTube API

Author: SteveJonesDev

src/pageScanner/rules/video-present.js

@@ -6,7 +6,7 @@
 
 export default {
 	id: 'video_present',
-	selector: 'video, iframe, object, source, [src], [role]',
+	selector: 'video, iframe, object, source, [src]:not(script), [role]',
 	excludeHidden: false,
 	tags: [
 		'wcag2a',

tests/jest/rules/videoElementPresent.test.js

@@ -135,6 +135,11 @@ describe( 'video_present rule', () => {
 			html: '<audio controls><source src="sound.mp3" type="audio/mpeg"></audio>',
 			shouldPass: true,
 		},
+		{
+			name: 'does not detect YouTube API script tag',
+			html: '<script type="text/javascript" src="https://www.youtube.com/iframe_api?ver=1.2.6" id="youtube-scripts-js"></script>',
+			shouldPass: true,
+		},
 	];
 
 	testCases.forEach( ( testCase ) => {