Fixed modal id implementation, added alternate reporter for e2e tests

frontend/src/metabase/components/ModalContent.jsx

@@ -15,7 +15,7 @@ export default class ModalContent extends Component {
 
     render() {
         return (
-            <div className={this.props.className}>
+            <div id={this.props.id} className={this.props.className}>
                 <div className="Modal-header Form-header flex align-center">
                     <h2 className="flex-full">{this.props.title}</h2>
                     <a className="text-grey-3 p1" onClick={this.props.closeFn}>

frontend/test/e2e/admin/datamodel.spec.js

 import path from "path";
 
+// FIXME: there are probably better places to put this
+import specReporter from "jasmine-spec-reporter";
+jasmine.getEnv().addReporter(new specReporter);
+
 import { isReady } from "../support/start-server";
 import { setup, cleanup } from "../support/setup";
 import { By, until } from "selenium-webdriver";

package.json

@@ -97,6 +97,7 @@
     "jasmine": "^2.4.1",
     "jasmine-core": "^2.4.1",
     "jasmine-promises": "^0.4.1",
+    "jasmine-spec-reporter": "^2.7.0",
     "json-loader": "^0.5.4",
     "karma": "^0.13.9",
     "karma-chrome-launcher": "^0.2.0",