garrensmith commented on a change in pull request #1139: Update
fauxton/notifications to use redux
URL: https://github.com/apache/couchdb-fauxton/pull/1139#discussion_r225107785
##########
File path:
app/addons/fauxton/notifications/__tests__/permanentNotification.test.js
##########
@@ -9,44 +9,54 @@
// WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
// License for the specific language governing permissions and limitations
under
// the License.
-import { PermanentNotification } from "../notifications";
-import Stores from "../stores";
-import FauxtonAPI from "../../../../core/api";
-import ActionTypes from "../actiontypes";
-import { mount } from "enzyme";
-import React from "react";
-import ReactDOM from "react-dom";
-
-const store = Stores.notificationStore;
+import { mount } from 'enzyme';
+import React from 'react';
+import thunk from 'redux-thunk';
+import { Provider } from 'react-redux';
+// import FauxtonAPI from '../../../../core/api';
Review comment:
You can remove
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services