275f041fbb
contentcollector: Simplify child node access
2021-01-27 04:59:36 +00:00
1cb5453aeb
contentcollector: Skip over non-Text, non-Element Nodes
2021-01-27 04:59:36 +00:00
075969aea0
contentcollector: Fix Element tag name fetch
...
The `name` property is only available on cheerio's Element-like
objects; DOM Element objects do not have a `name` property. Switch to
`dom.tagName()` to fix the logic for browsers.
2021-01-27 04:59:36 +00:00
e3ec9d9a4c
contentcollector: Fix parent node access
...
The `parent` property is only available on cheerio's Node-like
objects; DOM Node objects do not have a `parent` property. Switch to
the `parentNode` property so that the code works in browsers as well
as cheerio.
2021-01-27 04:59:36 +00:00
1d36549152
contentcollector: Delete unnecessary parentheses
2021-01-27 04:59:36 +00:00
e3a47e48f9
contentcollector: Fix collectContentLineText hook
...
Before, the hook always ignored the return values provided by the hook
functions. Now the hook functions can change the text by either
returning a string or setting `context.text` to the desired value.
Also drop the `styl` and `cls` context properties. They were never
documented and they were always null.
2021-01-27 04:59:36 +00:00
4e220538a1
contentcollector: Use destructuring to improve readability
2021-01-27 04:59:36 +00:00
b547ce9a47
contentcollector: Invert logic to improve readability
2021-01-27 04:59:36 +00:00
b811030846
contentcollector: Delete unnecessary truthiness check
2021-01-27 04:59:36 +00:00
fc2420c244
contentcollector: Fix iteration over child Nodes
...
In the DOM, `.children` only includes children that are Element
objects. In cheerio 0.22.0, `.children` includes all child Nodes, not
just Elements. Use `dom.numChildNodes()` and `dom.childNode()` so that
browsers behave the same as cheerio.
2021-01-27 04:59:36 +00:00
d0bfb54c0a
contentcollector: Avoid `for..in` iteration of object properties
...
`for..in` iterates over inherited properties, which is almost never
desired. In most cases there aren't any inherited enumerable
properties so it's not that big of a deal, but in the case of
HTMLCollection it's very bad because it iterates over every entry
twice (once by numerical index and once by name) plus it includes the
`length` property in the iteration.
2021-01-27 04:59:36 +00:00
3cfec58948
contentcollector: Rename `dom` functions for consistency with DOM spec
2021-01-27 04:59:36 +00:00
8763c3bb29
contentcollector: Fix Element attribute accesses
...
The `attribs` property is only available on cheerio's Element-like
objects; DOM Element objects do not have an `attribs` property. Switch
to `dom.nodeAttr()` to fix the logic for browsers.
2021-01-27 04:59:36 +00:00
99625950c8
contentcollector: Factor out call to `.toLowerCase()`
2021-01-27 04:59:36 +00:00
dd7fb1babe
contentcollector: Document the `dom` object
2021-01-27 04:59:36 +00:00
74bb2f76cc
contentcollector: Delete unused `domInterface` parameter
2021-01-27 04:59:36 +00:00
42c25b2536
openapi: Fix error logging
2021-01-27 04:59:36 +00:00
56f617060a
tests: Fix missing call to `done` callback
2021-01-27 04:59:36 +00:00
b164f9b431
tests: Replace "expected" with "want", "received" with "got"
...
"Got" and "want" are common terms for testing, plus this fixes a
spelling mistake ("received" was misspelled as "recieved").
2021-01-27 04:59:36 +00:00
fc69ae78aa
tests: Use `assert.deepEqual()` to simplify equality checks
2021-01-27 04:59:36 +00:00
dd815892f2
tests: Delete erroneous `describe()` calls
...
`describe()` is meant to be used by independent tests, but the tests
in this file are not independent. Add a higher-level `describe()` call
and delete all of the `describe()` calls that wrap a single test.
2021-01-27 04:59:36 +00:00
32a0df4883
tests: Fix invalid HTML in contentcollector tests
...
The HTML spec does not permit `<pre>` as a child of `<p>`.
2021-01-27 04:59:36 +00:00
53160f4a21
tests: Delete invalid contentcollector test
...
The HTML spec doesn't allow `<ul>` to be a child of `<ol>` (it must be
a child of `<li>` instead).
2021-01-27 04:59:36 +00:00
906b2624ed
tests: Re-enable import/export test that is now working
2021-01-27 04:59:36 +00:00
54a3dbb9a0
lint: Fix some straightforward ESLint errors
2021-01-27 04:59:36 +00:00
edf1dabe65
low hanging lint and some hard coded timeouts
2021-01-27 04:53:28 +00:00
809670dbea
derp
2021-01-26 21:42:52 +00:00
986894f776
derp
2021-01-26 21:28:19 +00:00
ff708c7160
derp
2021-01-26 20:00:14 +00:00
d8ec9309d5
derp
2021-01-26 19:57:01 +00:00
f6285e10e0
derp
2021-01-26 19:53:19 +00:00
d74782c7e6
make mocha show duration and include the first timeout value in a test
2021-01-26 19:42:23 +00:00
f2b54af6a4
testing 4 runners, will it be too much
2021-01-26 19:19:15 +00:00
b12becb6fb
testing 4 runners, will it be too much
2021-01-26 19:18:27 +00:00
f3677feba1
testing 4 runners, will it be too much
2021-01-26 18:55:16 +00:00
1b74f29f7b
PL
2021-01-26 17:46:04 +00:00
45d54ca874
perf test
2021-01-26 16:50:04 +00:00
48ebc3af5a
perf test
2021-01-26 16:48:08 +00:00
49ebba4d6c
some duration storage in stats
2021-01-26 16:41:08 +00:00
6a93c66de4
script to get each timing - usage bin/getTestRunnerTimings.sh
2021-01-26 13:48:12 +00:00
697bfd979b
use reporter
2021-01-26 11:09:22 +00:00
4b4b685bba
fix: runtime error if no buttons are present in toolbar ( #4680 )
...
Co-authored-by: Bartek Witczak <bartek@dayone.pl>
2021-01-26 11:02:54 +00:00
b73b0bcb98
farbtastic: Minimize diff to upstream
...
This should make it easier to upgrade to the latest version.
2021-01-26 04:07:43 -05:00
a0745d74b9
farbtastic: Document where the code came from
2021-01-26 04:07:43 -05:00
3a19254f21
stale code: removed excanvas which was ie support for no canvas
2021-01-26 04:05:54 -05:00
7768871f8f
security: bumping socketio version due to vulnerability
2021-01-26 00:53:04 -05:00
81b860bc35
tests: allow for longer timeout
2021-01-25 22:53:11 -05:00
44c2bc040c
lint: tests/backend/specs/api/tidy.js
2021-01-25 22:53:11 -05:00
610326b496
lint: tests/backend/specs/api/importexportGetPost.js
2021-01-25 22:53:11 -05:00
7421730b44
lint: src/node/utils/toolbar.js
2021-01-25 22:53:11 -05:00
Richard Hansen