Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Fix unit tests for non-english locales #8995

Merged
merged 2 commits into from
Oct 10, 2014

Conversation

marcelgerber
Copy link
Contributor

I finally found a centralized fix for all the issues with Strings in unit tests.
For #8672.

…cale

Conflicts:
	test/spec/FindReplace-test.js
@marcelgerber
Copy link
Contributor Author

Resolved conflicts.

@JeffryBooher
Copy link
Contributor

I can confirm that the function works but I haven't run the tests in another language since I have no language packs installed.

@JeffryBooher
Copy link
Contributor

I'll try again later on my Mac

@marcelgerber
Copy link
Contributor Author

@JeffryBooher You don't have to change the OS language, just change Brackets' language via Debug > Switch language.

@JeffryBooher
Copy link
Contributor

I did but I wanted to test with the default OS locale being something other than English. I tested this on Mac Japanese -- change looks good.

JeffryBooher added a commit that referenced this pull request Oct 10, 2014
Fix unit tests for non-english locales
@JeffryBooher JeffryBooher merged commit 72001b0 into adobe:master Oct 10, 2014
@marcelgerber marcelgerber deleted the test-window-en-locale branch October 10, 2014 19:05
@marcelgerber
Copy link
Contributor Author

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants