-
Notifications
You must be signed in to change notification settings - Fork 674
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Incorrect callstack for test error #1226
Comments
Lines marked with exclamation point are related to asyncToRegenerator calls (it replaces 'use strict';var _promise = require('D:\\Work\\testcafe-phoenix\\node_modules\\babel-runtime\\core-js\\promise.js');var _promise2 = _interopRequireDefault(_promise);var _help = require('./help.js');var _help2 = _interopRequireDefault(_help);function _interopRequireDefault(obj) {return obj && obj.__esModule ? obj : { default: obj };}function _asyncToGenerator(fn) {return function () {var gen = fn.apply(this, arguments);return new _promise2.default(function (resolve, reject) {function step(key, arg) {try {var info = gen[key](arg);var value = info.value;} catch (error) {reject(error);return;}if (info.done) {resolve(value);} else {return _promise2.default.resolve(value).then(function (value) {step("next", value);}, function (err) {step("throw", err);});}}return step("next");});};}
fixture`Fixture`.
page`http://devexpress.github.io/testcafe/example/`;
test('test', (() => {var _ref = _asyncToGenerator(function* (t) {
yield (0, _help2.default)(t);
});return function (_x) {return _ref.apply(this, arguments);};})()); @inikulin says, that it should be imported, not placed in the code. |
During testing approach, which @inikulin recommended, i've found that odd lines still here:
It's due to the fact, that transpilled |
@georgiy-abbasov Which of them are odd? If you are talking about core-js stuff - yes, they should be filtered out by stack filter. Debug to figure out why they still where. |
This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests. We recommend you ask TestCafe API, usage and configuration inquiries on StackOverflow. |
…ss#1226) (DevExpress#1236) * Changed babel options to avoid extra lines in report (closes DevExpress#1226) * Approach changed, added regression test for DevExpressgh-969 * another one approach * One more approach * Remarks fixed
Are you requesting a feature or reporting a bug?
bug
What is the current behavior?
What is the expected behavior?
How would you reproduce the current behavior (if this is a bug)?
run the test
Provide the test code and the tested page URL (if applicable)
Tested page URL: any
Test code
Specify your
7.4.0
,6.9.1
,0.10.47
The text was updated successfully, but these errors were encountered: