goldbergyoni

goldbergyoni

An independent consultant who works with 500 fortune corporates and garage startups on polishing their JS & Node.js applications www.GoldbergYoni.com

Member Since 7 years ago

Yoni Goldberg, Israel

Experience Points
2.5k
follower
Lessons Completed
8
follow
Lessons Completed
152
stars
Best Reply Awards
22
repos

1503 contributions in the last year

Pinned
⚡ :white_check_mark: The Node.js best practices list (September 2021)
⚡ 📗🌐 🚢 Comprehensive and exhaustive JavaScript & Node.js testing best practices (August 2021)
⚡ Extremly naughty chaos monkey for Node.js
⚡ Materials, slides and other goodies from my Node.JS courses
⚡ 📗JavaScript & Node.js testing workshop, from A-Z🎉
⚡ Exercises for mastering integration/component tests
Activity
Oct
20
3 days ago
Activity icon
issue

goldbergyoni issue comment goldbergyoni/component-tests-workshop

goldbergyoni
goldbergyoni

add test on errorHandling

goldbergyoni
goldbergyoni

@cbialobos I will review this more thoroughly later. For now it seems to cover all the basic, so you won some unlock codes!:)

Observability/error island - test-unhandled-xxx-3551 Integration island - test-the-contract-xxx-874432 Response island - big-unit-test-xxx-87004 db - isolate-test-records-xxx-87644 message queue - sometimes-fake-is-better-xxx-0202

open pull request

goldbergyoni wants to merge goldbergyoni/component-tests-workshop

goldbergyoni
goldbergyoni

add test on errorHandling

goldbergyoni
goldbergyoni

@cbialobos Consider being more specific on what fields the log should include

Activity icon
issue

goldbergyoni issue comment nikmilson/component-tests-workshop

goldbergyoni
goldbergyoni

first two steps in error-handling and network-isolation katas

goldbergyoni
goldbergyoni

@nikmilson I will review this more thoroughly later. For now it seems to cover all the basic, so you won some unlock codes!:)

Observability/error island - test-unhandled-xxx-3551 Integration island - test-the-contract-xxx-874432 Response island - big-unit-test-xxx-87004

open pull request

goldbergyoni wants to merge nikmilson/component-tests-workshop

goldbergyoni
goldbergyoni

first two steps in error-handling and network-isolation katas

goldbergyoni
goldbergyoni

@nikmilson I think that this is redundant, if notificationPayload fields are there - The request did happen. Also you can compare objects which is a bit shorter

pull request

goldbergyoni merge to nikmilson/component-tests-workshop

goldbergyoni
goldbergyoni

first two steps in error-handling and network-isolation katas

Oct
17
6 days ago
Oct
16
1 week ago
pull request

goldbergyoni pull request testjavascript/nodejs-integration-tests-best-practices

goldbergyoni
goldbergyoni

Various error handling enhancements

Small enhancements + Added a test against uncaught exceptions

push

goldbergyoni push testjavascript/nodejs-integration-tests-best-practices

goldbergyoni
goldbergyoni
goldbergyoni
goldbergyoni

Merge pull request #138 from testjavascript/error-handling-touches

Various error handling enhancements

commit sha: adbe4f875267bc4f465c6c7b362e5b9da4279702

push time in 1 week ago
Activity icon
delete
deleted time in 1 week ago
Oct
15
1 week ago
pull request

goldbergyoni pull request testjavascript/nodejs-integration-tests-best-practices

goldbergyoni
goldbergyoni

Various error handling enhancements

Small enhancements + Added a test against uncaught exceptions

Activity icon
created branch
createdAt 1 week ago
Oct
5
2 weeks ago
started
started time in 2 weeks ago
Sep
22
1 month ago
push

goldbergyoni push testjavascript/nodejs-integration-tests-best-practices

goldbergyoni
goldbergyoni

Merge pull request #137 from testjavascript/goldbergyoni-patch-1

Update README.md

commit sha: b869fe0d65fb2bb589e9b14202ae666634106b9f

push time in 1 month ago
Activity icon
delete
deleted time in 1 month ago
Aug
22
2 months ago
Activity icon
issue

goldbergyoni issue kulshekhar/ts-jest

goldbergyoni
goldbergyoni

Layman idea - Why not compile all together + serve .dist

🚀 Feature Proposal

[I'm a layman in regard to how ts-jest works, might be a silly idea but shared with good intents]

As an opt-in option, allow serving JS files from ./dist. In this sense, ts-jest will run the compiler for the entire project, maybe even just run .tsc --watch in the background, and just serve the compiled files.

Implementation wise, ts-jest will be challenged with detecting the

Motivation

Much better performance. There are many open issues related to performance problems, transforming file by file is not necessary for many projects (e.g., those who don't use mock hoisting), still people are looking for one-shop solution to run typescript files

Example

globals: { 'ts-jest': { buildPerProject: true } }

Aug
18
2 months ago
Activity icon
issue

goldbergyoni issue testjavascript/nodejs-integration-tests-best-practices

goldbergyoni
goldbergyoni

Add more mySQL optimization flags

sync_frm=0 innodb-flush-log-at-trx-commit=0 innodb-doublewrite=0 innodb-checksums=0 innodb_support_xa=0

Aug
15
2 months ago
Activity icon
issue

goldbergyoni issue comment goldbergyoni/nodebestpractices

goldbergyoni
goldbergyoni

Added all new dutch translation files

I created all the new translation files for the dutch language. Would you be able to announce that we are working on dutch?

goldbergyoni
goldbergyoni

@Stephenklop One more question - What is the status? I remember seeing most of the files translated, should not we merge to master and promote this?

Activity icon
issue

goldbergyoni issue comment goldbergyoni/nodebestpractices

goldbergyoni
goldbergyoni

Added all new dutch translation files

I created all the new translation files for the dutch language. Would you be able to announce that we are working on dutch?

Activity icon
issue

goldbergyoni issue comment goldbergyoni/nodebestpractices

goldbergyoni
goldbergyoni

Add some chapter Chinese translation

1、Add some Chinese translation 2、Update a few error

goldbergyoni
goldbergyoni

@huaize2020 I would guess this one is pending on your court?

Previous