Removed inappropriate section tags. Because screen readers announce each section, they should not be used unless they are actually defining meaningful sections.

Added feedback check for AI-assessment test

Added peer assessment feedback test

Refactored stub services for style and DRY

Added unit tests for stub implementations

Updated acceptance tests that depend on stubs.

Updated Studio acceptance tests to use YouTube stub server; fixed failing tests in devstack.

Added wait for JS on upload page
Fixed an issue where css_click wasn't waiting for JS to load

Cleaned up files with muliline imports

Cleaned up files that do not use these imports

Misread comment

Instead, we use XModule field default values when creating an empty
XModule. Driven by this use case, we also allow for XModules to be
created in memory without being persisted to the database at all. This
necessitates a change to the Modulestore api, replacing clone_item with
create_draft and save_xmodule.

I accidently had show_answer instead of showanswer.  This error was hidden by a previous default of showanswer=always.

New function was added: is_css_not_present
This function works like is_css_present in that it will wait and can take in an optional argument to wait longer.  This should be used everywhere INSTEAD of not is_css_present as in the latter case, you are telling selenium to wait for the thing you don't want to be there to either be there or time out.

* Missing docstring
* Redefining name from outer scope

for common ui manipulations

an "unanswered" div or span.  This is much faster than
asserting that correct/incorrect is NOT present.

world.css_click() helper instead of directly
calling splinter.