1247052 - Improper outdated octal constant syntax in mailnews/compose/test/unit/test_bug235432.js

Status: RESOLVED FIXED

(Thunderbird :: Testing Infrastructure, defect)

Description

[ISHIKAWA, Chiaki]

File mailnews/compose/test/unit/test_bug235432.js
has an outdated octal constant format on line 80 0600 as opposed to proper 0o600 

This results in warning/syntax error during C-C TB's xpcshell-tests run.

This should be fixed to remove any clutter from the log.

Comment 1

[ISHIKAWA, Chiaki]

Attachment: Use proper octal constant syntax.

Here is the patch.

BTW, the warning shows up in the log of Windows exceution run of xpcshell-tests on try-comm-central as follows: very verbose.

08:12:12     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "SyntaxError: octal literals and octal escape sequences are deprecated" {file: "C:/slave/test/build/tests/xpcshell/tests/mailnews/compose/test/unit/test_bug235432.js" line: 80 column: 51 source: "    file.createUnique(Ci.nsIFile.NORMAL_FILE_TYPE, 0600);
08:12:12     INFO -  "}]"
08:12:12     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "SyntaxError: octal literals and octal escape sequences are deprecated" {file: "C:/slave/test/build/tests/xpcshell/tests/mailnews/compose/test/unit/test_bug235432.js" line: 80 column: 51 source: "    file.createUnique(Ci.nsIFile.NORMAL_FILE_TYPE, 0600);
08:12:12     INFO -  "}]"
08:12:12     INFO -  "CONSOLE_MESSAGE: (warn) [JavaScript Warning: "SyntaxError: octal literals and octal escape sequences are deprecated" {file: "C:/slave/test/build/tests/xpcshell/tests/mailnews/compose/test/unit/test_bug235432.js" line: 80 column: 51 source: "    file.createUnique(Ci.nsIFile.NORMAL_FILE_TYPE, 0600);

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 8717601 [details] [diff] [review]
Use proper octal constant syntax.

Sorry, I'm no long actively working on Thunderbird.

Comment 3

[Kent James (:rkent)]

Comment on attachment 8717601 [details] [diff] [review]
Use proper octal constant syntax.

Review of attachment 8717601 [details] [diff] [review]:
-----------------------------------------------------------------

Stealing this trivial review.

This is fine (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Numbers_and_dates)

There are a few other places this occurs in tests, you might fix those as well, preferably in a single bug.

Comment 4

[ISHIKAWA, Chiaki]

OK I will o(In reply to Mark Banner (:standard8) from comment #2)
> Comment on attachment 8717601 [details] [diff] [review]
> Use proper octal constant syntax.
> 
> Sorry, I'm no long actively working on Thunderbird.

Sorry, you name came on the recommeded reviewer list and you had the smallest # of review requests. I bear this in mind, though, in the future.

(In reply to Kent James (:rkent) from comment #3)
> Comment on attachment 8717601 [details] [diff] [review]
> Use proper octal constant syntax.
> 
> Review of attachment 8717601 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Stealing this trivial review.
> 
> This is fine
> (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/
> Numbers_and_dates)
> 
> There are a few other places this occurs in tests, you might fix those as
> well, preferably in a single bug.

I will look for them and creates a one patch, then.
spcshell-test(s) only print the logs when a unit test fails. So there are probably 
many instances, but testers were not aware of the improper syntax before
because the logs were not printed at all.


TIA

Comment 5

[ISHIKAWA, Chiaki]

I am creating another bugzilla for the fixing of other usages in the tree.
I would put a coule of patches there: one for C-C portion and the other for M-C portion.

I am putting checkin-needed keyword for this entry.

Comment 6

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/fe0300249b7d125c87448b522c178c539aefaadb
Bug 1247052 - Use proper octal constant syntax in test_bug235432.js. r=rkent

Comment 7

[Jorg K (CEST = GMT+2)]

Comment on attachment 8717601 [details] [diff] [review]
Use proper octal constant syntax.

This is the perfect patch to push to Aurora (TB 46) to get the build there going (since I foolishly had checked in some stuff with DONTBUILD).
https://hg.mozilla.org/releases/comm-aurora/rev/6853aa433495
It does absolutely no harm. Sorry guys to hijack this for my purposes ;-)

Comment 8

[Kent James (:rkent)]

Jorg, you need the self-service build page to allow you to start builds:

https://secure.pub.build.mozilla.org/buildapi/self-serve

If you don't have permissions for that, file a bug asking for them.

Comment 9

[Jorg K (CEST = GMT+2)]

Thanks, I do have permissions for that page, the normal LDAP permissions suffice. However, what do I do? At the bottom the page says:
Create new dep builds on comm-aurora revision
Create new PGO builds on comm-aurora revision
Create new nightly builds on comm-aurora revision
The three lines have an input field and a "Submit" button.
So I enter the revision I want in the last line for the nightly build and press submit?

Comment 10

[Kent James (:rkent)]

(In reply to Jorg K (GMT+1) from comment #9)

> So I enter the revision I want in the last line for the nightly build and
> press submit?

Basically, yes. Starting a job is also very low impact, so don't be afraid to try things.

Comment 11

[rsx11m]

Cancelling checkin-needed as this already landed on comm-aurora.

Comment 12

[Jorg K (CEST = GMT+2)]

checkin-needed is for comm-central. It was landed there, see comment #6. Aurora landing in comment #7.