Revert "Merge pull request #13756 from NejcZdovc/fix/#13754-regex"

This reverts commit 06c774b.

app/cmdLine.js

@@ -17,6 +17,7 @@ const fs = require('fs')
 const path = require('path')
 
 const isDarwin = process.platform === 'darwin'
+const promoCodeFilenameRegex = /-([a-zA-Z\d]{3}\d{3})\s?(?:\(\d+\))?$/g
 const debugTabEventsFlagName = '--debug-tab-events'
 
 let appInitialized = false
@@ -165,9 +166,8 @@ const api = module.exports = {
     // parse promo code from installer path
     // first, get filename
     const fileName = path.win32.parse(installerPath).name
-    const promoCodeFilenameRegex = /-(([a-zA-Z\d]{3}\d{3})|([a-zA-Z]{1,}-[a-zA-Z]{1,}))\s?(?:\(\d+\))?$/g
     const matches = promoCodeFilenameRegex.exec(fileName)
-    if (matches && matches.length > 1) {
+    if (matches && matches.length === 2) {
       return matches[1]
     }
     return null

build/pkg-scripts/postinstall

@@ -8,7 +8,7 @@ installationPath=$2
 # TODO: ugly to assume the name of the app, especially with multi-channel.
 # Luckily for now, release channel is the only channel with a pkg installer.
 installationAppPath="$installationPath/Brave.app"
-installerPathPromoCodeRegex='.+-(([a-zA-Z0-9]{3}[0-9]{3})|([a-zA-Z]{1,}-[a-zA-Z]{1,}))([[:blank:]]?\([0-9]+\))?\.pkg$'
+installerPathPromoCodeRegex='.+-([a-zA-Z0-9]{3}[0-9]{3})([[:blank:]]?\([0-9]+\))?\.pkg$'
 userDataDir="$HOME/Library/Application Support/brave"
 promoCodePath="$userDataDir/promoCode"
 # pkg runs this script as root so we need to get current username

test/unit/app/cmdLineTest.js

@@ -52,31 +52,6 @@ describe('cmdLine', function () {
       const result = this.cmdLine.getFirstRunPromoCode(args)
       assert.equal(result, promoCode)
     })
-
-    it('finds and parses promo code when we have multiple downloads', function () {
-      const promoCode = 'pem001 (1)'
-      const validPromoCodeInstallerPath = `d:\\my\\location\\on-disk\\in-a-folder-tes301\\Setup-Brave-x64-${promoCode}.exe`
-      const args = [...initialArgs, key, validPromoCodeInstallerPath, '--other', 'arg', '--and-another']
-      const result = this.cmdLine.getFirstRunPromoCode(args)
-      assert.equal(result, 'pem001')
-    })
-
-    it('finds and parses promo code 2', function () {
-      const promoCode = 'org-name'
-      const validPromoCodeInstallerPath = `d:\\my\\location\\on-disk\\in-a-folder-tes301\\Setup-Brave-x64-${promoCode}.exe`
-      const args = [...initialArgs, key, validPromoCodeInstallerPath, '--other', 'arg', '--and-another']
-      const result = this.cmdLine.getFirstRunPromoCode(args)
-      assert.equal(result, promoCode)
-    })
-
-    it('finds and parses promo code 2 when we have multiple downloads', function () {
-      const promoCode = 'org-name (1)'
-      const validPromoCodeInstallerPath = `d:\\my\\location\\on-disk\\in-a-folder-tes301\\Setup-Brave-x64-${promoCode}.exe`
-      const args = [...initialArgs, key, validPromoCodeInstallerPath, '--other', 'arg', '--and-another']
-      const result = this.cmdLine.getFirstRunPromoCode(args)
-      assert.equal(result, 'org-name')
-    })
-
     it(`does not find promo code when there isn't one`, function () {
       const noPromoCodeInstallerPath = `d:\\my\\location\\on-disk\\in-a-folder-tes301\\Setup-Brave-x64.exe`
       const args = [...initialArgs, key, noPromoCodeInstallerPath, '--other', 'arg', '--and-another']