FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

update unit test example by Stwissel · Pull Request #329 · node-red/node-red.github.io · GitHub

/ node-red.github.io Public

update unit test example - #329

Open
Stwissel wants to merge 2 commits into
node-red:mainfrom
Stwissel:unit-test-update
ConversationCommits (2)ChecksFiles changed
Open

update unit test example#329
Stwissel wants to merge 2 commits into
node-red:mainfrom
Stwissel:unit-test-update

Conversation

Stwissel commented Nov 9, 2023 •
edited
Loading

Copy link
Copy Markdown

Got confused why unit test didn't work, learned the documentation is outdated.
I'd like to save others that learning experience :-)

Fixes #328

(formatting changes by pretttierJS sorry - hope it isn't an issue)

update unit test example
ca6e56b 
(formatting cganges by pretttier)

linux-foundation-easycla Bot commented Nov 9, 2023 •
edited
Loading

Copy link
Copy Markdown



The committers listed above are authorized under a signed CLA.

Stwissel commented Oct 23, 2024

Copy link
Copy Markdown
Author

@knolleary - merge this?

dceejay commented Oct 23, 2024

Copy link
Copy Markdown
Member

Hi - sorry - there seem to be lots of formatting changes that aren't actual changes - That make it hard to spot the real changes... any chance you can undo them ?

knolleary commented Oct 23, 2024

Copy link
Copy Markdown
Member

TBH, no problem with the formatting changes - they are all sensible. The bit that caused me to pause previously was adding node-red as a full dev dependency. That doesn't feel optimal, but I'm not sure how to describe the alternatives better in the docs. Will noodle on it some more.

Stwissel commented Oct 30, 2024

Copy link
Copy Markdown
Author

I'm curious. The test helper depends on nodeRED. How to get around it?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unit testing sample code doesn't match test-helper

3 participants

Back | FazBrowse Home | New Git URL