Change BankAccount#statusLine to report ambiguous bank accounts

Author: dkubb

spec/bank_account_report_spec.js

@@ -53,7 +53,7 @@ describe('BankAccountReport', () => {
     it('returns the report', () => {
       expect(report.generate()).toEqual([
         '000000051\n',
-        '664371495 ERR\n',
+        '664371485\n',
         '49006771? ILL\n',
         '1234?678? ILL\n'
       ])

spec/bank_account_spec.js

@@ -331,6 +331,22 @@ describe('BankAccount', () => {
         expect(bankAccount.statusLine()).toBe('345882864 ERR\n')
       })
     })
+
+    describe('when the bank account is ambiguous', () => {
+      const text = [
+        ' _  _  _  _  _  _  _  _  _ ',
+        '|_||_||_||_||_||_||_||_||_|',
+        '|_||_||_||_||_||_||_||_||_|'
+      ]
+
+      const bankAccount = BankAccount.parse(text.join('\n'))
+
+      it('returns the ambiguous status', () => {
+        expect(bankAccount.statusLine()).toBe(
+          "888888888 AMB ['888886888', '888888880', '888888988']\n"
+        )
+      })
+    })
   })
 
   describe('#alternatives', () => {

src/bank_account.js

@@ -123,10 +123,17 @@ class BankAccount {
    * @returns {string}
    */
   statusLine () {
-    // Display illegible or error status. If the Bank Account
+    const alternatives = this.alternatives()
+    if (alternatives.length === 1) return alternatives[0].statusLine()
+
+    const ambiguous = alternatives.map(ba => `'${ba.format()}'`).join(', ')
+
+    // If there are ambiguous Bank Account numbers return them,
+    // otherwise display illegible or error status. If the Bank Account
     // is valid, then return no status.
-    const status = !this.isLegible() ? ' ILL' :
-                   !this.isValid()   ? ' ERR' :
+    const status = ambiguous         ? ` AMB [${ambiguous}]` :
+                   !this.isLegible() ? ' ILL'                :
+                   !this.isValid()   ? ' ERR'                :
                                        ''
 
     return `${this.format()}${status}\n`