Paths

Table of Contentst

Differential D14511

[apps] Consistently apply naming convention for mocha unit test files
ClosedPublic
Actions

Authored by bytesofman on Sep 20 2023, 18:25.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC6991ac9fccd4: [apps] Consistently apply naming convention for mocha unit test files

Summary

Using '.test.js' is picked up by VScode's GUI and imo is easier to parse as "this file is the test file for the other file with the same name without the '.test'" -- esp vs the camelCase _Tests everywhere

Test Plan

Verify only change is file names
npm test

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Sep 20 2023, 18:25

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 20 2023, 18:25

bytesofman requested review of this revision.Sep 20 2023, 18:25

Harbormaster completed remote builds in B25105: Diff 42304.Sep 20 2023, 18:37

Fabien accepted this revision.Sep 21 2023, 07:19

This revision is now accepted and ready to land.Sep 21 2023, 07:19

Closed by commit rABC6991ac9fccd4: [apps] Consistently apply naming convention for mocha unit test files (authored by bytesofman). · Explain WhySep 21 2023, 12:15

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC6991ac9fccd4: [apps] Consistently apply naming convention for mocha unit test files.

Revision Contents
Changeset List

Path

Size

apps/

alias-server/

test/

	alias.test.js
	aliasTests.js

aliasTests.js

	chronik.test.js
	chronikTests.js

chronikTests.js

	chronikWsHandler.test.js
	chronikWsHandlerTests.js

chronikWsHandlerTests.js

	db.test.js
	dbTests.js

dbTests.js

	events.test.js
	eventsTests.js

eventsTests.js

	main.test.js
	mainTests.js

mainTests.js

	rpc.test.js
	rpcTests.js

rpcTests.js

	script.test.js
	scriptTests.js

scriptTests.js

	telegram.test.js
	telegramTests.js

telegramTests.js

	utils.test.js
	utilsTests.js

utilsTests.js

ecash-herald/

test/

	chronik.test.js
	chronikTests.js

chronikTests.js

	chronikWsHandler.test.js
	chronikWsHandlerTests.js

chronikWsHandlerTests.js

	events.test.js
	eventsTests.js

eventsTests.js

	main.test.js
	mainTests.js

mainTests.js

	parse.test.js
	parseTests.js

parseTests.js

	telegram.test.js
	telegramTests.js

telegramTests.js

	utils.test.js
	utilsTests.js

utilsTests.js

Diff 42316

View Options

apps/alias-server/test/alias.test.js

View Options

apps/alias-server/test/aliasTests.js

View Options

apps/alias-server/test/chronik.test.js

View Options

apps/alias-server/test/chronikTests.js

View Options

apps/alias-server/test/chronikWsHandler.test.js

View Options

apps/alias-server/test/chronikWsHandlerTests.js

View Options

apps/alias-server/test/db.test.js

View Options

apps/alias-server/test/dbTests.js

View Options

apps/alias-server/test/events.test.js

View Options

apps/alias-server/test/eventsTests.js

View Options

apps/alias-server/test/main.test.js

View Options

apps/alias-server/test/mainTests.js

View Options

apps/alias-server/test/rpc.test.js

View Options

apps/alias-server/test/rpcTests.js

View Options

apps/alias-server/test/script.test.js

View Options

apps/alias-server/test/scriptTests.js

View Options

apps/alias-server/test/telegram.test.js

View Options

apps/alias-server/test/telegramTests.js

View Options

apps/alias-server/test/utils.test.js

View Options

apps/alias-server/test/utilsTests.js

View Options

apps/ecash-herald/test/chronik.test.js

View Options

apps/ecash-herald/test/chronikTests.js

View Options

apps/ecash-herald/test/chronikWsHandler.test.js

View Options

apps/ecash-herald/test/chronikWsHandlerTests.js

View Options

apps/ecash-herald/test/events.test.js

View Options

apps/ecash-herald/test/eventsTests.js

View Options

apps/ecash-herald/test/main.test.js

View Options

apps/ecash-herald/test/mainTests.js

View Options

apps/ecash-herald/test/parse.test.js

View Options

apps/ecash-herald/test/parseTests.js

View Options

apps/ecash-herald/test/telegram.test.js

View Options

apps/ecash-herald/test/telegramTests.js

View Options

apps/ecash-herald/test/utils.test.js

View Options

apps/ecash-herald/test/utilsTests.js

[apps] Consistently apply naming convention for mocha unit test filesClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 42316

apps/alias-server/test/alias.test.js

apps/alias-server/test/aliasTests.js

apps/alias-server/test/chronik.test.js

apps/alias-server/test/chronikTests.js

apps/alias-server/test/chronikWsHandler.test.js

apps/alias-server/test/chronikWsHandlerTests.js

apps/alias-server/test/db.test.js

apps/alias-server/test/dbTests.js

apps/alias-server/test/events.test.js

apps/alias-server/test/eventsTests.js

apps/alias-server/test/main.test.js

apps/alias-server/test/mainTests.js

apps/alias-server/test/rpc.test.js

apps/alias-server/test/rpcTests.js

apps/alias-server/test/script.test.js

apps/alias-server/test/scriptTests.js

apps/alias-server/test/telegram.test.js

apps/alias-server/test/telegramTests.js

apps/alias-server/test/utils.test.js

apps/alias-server/test/utilsTests.js

apps/ecash-herald/test/chronik.test.js

apps/ecash-herald/test/chronikTests.js

apps/ecash-herald/test/chronikWsHandler.test.js

apps/ecash-herald/test/chronikWsHandlerTests.js

apps/ecash-herald/test/events.test.js

apps/ecash-herald/test/eventsTests.js

apps/ecash-herald/test/main.test.js

apps/ecash-herald/test/mainTests.js

apps/ecash-herald/test/parse.test.js

apps/ecash-herald/test/parseTests.js

apps/ecash-herald/test/telegram.test.js

apps/ecash-herald/test/telegramTests.js

apps/ecash-herald/test/utils.test.js

apps/ecash-herald/test/utilsTests.js

[apps] Consistently apply naming convention for mocha unit test files
ClosedPublic
Actions

Revision Contents
Changeset List