Skip to content

Commit

Permalink
fix(tests): Enable --debug for test:compile:advanced; fix some errors (
Browse files Browse the repository at this point in the history
…google#5959)

* Enable the --debug flag when running the
  buildAdvancedCompilationTest gulp task.
* Migrate test/compile/main.js to goog.module.
  * Use more selective goog.requires.
    * Reduces compiled size from ~400k to ~370k.
  * @Suppress "extra" requires needed for side effects.
  • Loading branch information
cpcallen authored Mar 1, 2022
1 parent 9ba2571 commit 88334be
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 9 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
"test": "tests/run_all_tests.sh",
"test:generators": "tests/scripts/run_generators.sh",
"test:mocha:interactive": "http-server ./ -o /tests/mocha/index.html -c-1",
"test:compile:advanced": "gulp buildAdvancedCompilationTest",
"test:compile:advanced": "gulp buildAdvancedCompilationTest --debug",
"typings": "gulp typings",
"updateGithubPages": "gulp gitUpdateGithubPages"
},
Expand Down
30 changes: 22 additions & 8 deletions tests/compile/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,19 +4,33 @@
* SPDX-License-Identifier: Apache-2.0
*/

goog.provide('Main');
goog.module('Main');

// Core
// Either require 'Blockly.requires', or just the components you use:
goog.require('Blockly');
/* eslint-disable-next-line no-unused-vars */
const {BlocklyOptions} = goog.requireType('Blockly.BlocklyOptions');
const {inject} = goog.require('Blockly.inject');
/** @suppress {extraRequire} */
goog.require('Blockly.geras.Renderer');
/** @suppress {extraRequire} */
goog.require('Blockly.VerticalFlyout');
// Blocks
goog.require('Blockly.libraryBlocks');
/** @suppress {extraRequire} */
goog.require('Blockly.libraryBlocks.logic');
/** @suppress {extraRequire} */
goog.require('Blockly.libraryBlocks.loops');
/** @suppress {extraRequire} */
goog.require('Blockly.libraryBlocks.math');
/** @suppress {extraRequire} */
goog.require('Blockly.libraryBlocks.texts');
/** @suppress {extraRequire} */
goog.require('Blockly.libraryBlocks.testBlocks');

Main.init = function() {
Blockly.inject('blocklyDiv', {
'toolbox': document.getElementById('toolbox')
});

function init() {
inject('blocklyDiv', /** @type {BlocklyOptions} */ ({
'toolbox': document.getElementById('toolbox')
}));
};
window.addEventListener('load', Main.init);
window.addEventListener('load', init);

0 comments on commit 88334be

Please sign in to comment.