#!/usr/bin/env python3
# Copyright 2015 The Chromium Authors
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.

from . import js_checker
from os import path as os_path
import re
from sys import path as sys_path
from . import test_util
import unittest

_HERE = os_path.dirname(os_path.abspath(__file__))
sys_path.append(os_path.join(_HERE, "..", ".."))

from PRESUBMIT_test_mocks import MockInputApi, MockOutputApi


_DECLARATION_METHODS = "const", "let", "var"


class JsCheckerTest(unittest.TestCase):
  def setUp(self):
    super(JsCheckerTest, self).setUp()

    self.checker = js_checker.JSChecker(MockInputApi(), MockOutputApi())

  def ShouldFailBindThisCheck(self, line):
    error = self.checker.BindThisCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    highlight = test_util.GetHighlight(line, error).strip()
    self.assertTrue(highlight.startswith(".bind(this"))

  def ShouldPassBindThisCheck(self, line):
    self.assertEqual(
      "",
      self.checker.BindThisCheck(1, line),
      "Should not be flagged as style error: " + line,
    )

  def testBindThisFails(self):
    lines = [
      'this.api_.getThinger((function() {console.log(\'foo\');}).bind(this));',
      'this.api_.getThinger((function foo() {console.log(\'foo\');}).bind(this));',
    ]
    for line in lines:
      self.ShouldFailBindThisCheck(line)

  def testBindThisPasses(self):
    lines = [
      'let bound = this.method_.bind(this);',
      "document.addEventListener('click', this.onClick_.bind(this));",
      'this.api_.onEvent = this.onClick_.bind(this);',
      'this.api_.getThinger(this.gotThinger_.bind(this));',
      'this.api_.getThinger(this.gotThinger_.bind(this, param1, param2));',
      '// And in the darkness bind them.',
      'this.methodName_.bind(scope, param)',
    ]
    for line in lines:
      self.ShouldPassBindThisCheck(line)

  def ShouldFailCommentCheck(self, line):
    """Checks that uncommented "<if>" and "<include>" are a style error."""
    error = self.checker.CommentIfAndIncludeCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    highlight = test_util.GetHighlight(line, error).strip()
    self.assertTrue(highlight.startswith(("<if", "<include")))

  def ShouldPassCommentCheck(self, line):
    """Checks that commented "<if>" and "<include>" are allowed."""
    self.assertEqual(
      "",
      self.checker.CommentIfAndIncludeCheck(1, line),
      "Should not be flagged as style error: " + line,
    )

  def testCommentFails(self):
    lines = [
      '<include src="blah.js">',
      # Currently, only "// " is accepted (not just "//" or "//\s+") as Python
      # can't do variable-length lookbehind.
      '//<include src="blah.js">',
      '//  <include src="blah.js">',
      '             <include src="blee.js">',
      '  <if expr="chromeos">',
      '<if expr="lang == \'de\'">',
      '//<if expr="bitness == 64">',
    ]
    for line in lines:
      self.ShouldFailCommentCheck(line)

  def testCommentPasses(self):
    lines = [
      '// <include src="assert.js">',
      '             // <include src="util.js"/>',
      '// <if expr="chromeos">',
      '           // <if expr="not chromeos">',
      "   '<iframe src=blah.html>';",
    ]
    for line in lines:
      self.ShouldPassCommentCheck(line)

  def ShouldFailChromeSendCheck(self, line):
    """Checks that the "chrome.send" checker flags |line| as a style error."""
    error = self.checker.ChromeSendCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    self.assertEqual(test_util.GetHighlight(line, error), ", []")

  def ShouldPassChromeSendCheck(self, line):
    """Checks that the "chrome.send" checker doesn"t flag |line| as a style
    error.
    """
    self.assertEqual(
      "",
      self.checker.ChromeSendCheck(1, line),
      "Should not be flagged as style error: " + line,
    )

  def testChromeSendFails(self):
    lines = [
      "chrome.send('message', []);",
      "  chrome.send('message', []);",
    ]
    for line in lines:
      self.ShouldFailChromeSendCheck(line)

  def testChromeSendPasses(self):
    lines = [
      'chrome.send("message", constructArgs("foo", []));',
      '  chrome.send("message", constructArgs("foo", []));',
      'chrome.send("message", constructArgs([]));',
      '  chrome.send("message", constructArgs([]));',
    ]
    for line in lines:
      self.ShouldPassChromeSendCheck(line)

  def ShouldFailEndJsDocCommentCheck(self, line):
    """Checks that the **/ checker flags |line| as a style error."""
    error = self.checker.EndJsDocCommentCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    self.assertEqual(test_util.GetHighlight(line, error), "**/")

  def ShouldPassEndJsDocCommentCheck(self, line):
    """Checks that the **/ checker doesn"t flag |line| as a style error."""
    self.assertEqual(
      "",
      self.checker.EndJsDocCommentCheck(1, line),
      "Should not be flagged as style error: " + line,
    )

  def testEndJsDocCommentFails(self):
    lines = [
      "/** @override **/",
      "/** @type {number} @const **/",
      "  **/",
      "**/  ",
    ]
    for line in lines:
      self.ShouldFailEndJsDocCommentCheck(line)

  def testEndJsDocCommentPasses(self):
    lines = [
      "/***************/",  # visual separators
      "  */",  # valid JSDoc comment ends
      "*/  ",
      "/**/",  # funky multi-line comment enders
      "/** @override */",  # legit JSDoc one-liners
    ]
    for line in lines:
      self.ShouldPassEndJsDocCommentCheck(line)

  def ShouldFailExtraDotInGenericCheck(self, line):
    """Checks that Array.< or Object.< is flagged as a style nit."""
    error = self.checker.ExtraDotInGenericCheck(1, line)
    self.assertNotEqual("", error)
    self.assertTrue(test_util.GetHighlight(line, error).endswith(".<"))

  def testExtraDotInGenericFails(self):
    lines = [
      "/** @private {!Array.<!Frobber>} */",
      "var a = /** @type {Object.<number>} */({});",
      "* @return {!Promise.<Change>}",
    ]
    for line in lines:
      self.ShouldFailExtraDotInGenericCheck(line)

  def ShouldFailInheritDocCheck(self, line):
    """Checks that the "@inheritDoc" checker flags |line| as a style error."""
    error = self.checker.InheritDocCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    self.assertEqual(test_util.GetHighlight(line, error), "@inheritDoc")

  def ShouldPassInheritDocCheck(self, line):
    """Checks that the "@inheritDoc" checker doesn"t flag |line| as a style
    error.
    """
    self.assertEqual(
      "",
      self.checker.InheritDocCheck(1, line),
      msg="Should not be flagged as style error: " + line,
    )

  def testInheritDocFails(self):
    lines = [
      " /** @inheritDoc */",
      "   * @inheritDoc",
    ]
    for line in lines:
      self.ShouldFailInheritDocCheck(line)

  def testInheritDocPasses(self):
    lines = [
      "And then I said, but I won't @inheritDoc! Hahaha!",
      " If your dad's a doctor, do you inheritDoc?",
      "  What's up, inherit doc?",
      "   this.inheritDoc(someDoc)",
    ]
    for line in lines:
      self.ShouldPassInheritDocCheck(line)

  def ShouldFailPolymerLocalIdCheck(self, line):
    """Checks that element.$.localId check marks |line| as a style error."""
    error = self.checker.PolymerLocalIdCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    self.assertTrue(".$" in test_util.GetHighlight(line, error))

  def ShouldPassPolymerLocalIdCheck(self, line):
    """Checks that element.$.localId check doesn't mark |line| as a style
    error."""
    self.assertEqual(
      "",
      self.checker.PolymerLocalIdCheck(1, line),
      msg="Should not be flagged as a style error: " + line,
    )

  def testPolymerLocalIdFails(self):
    lines = [
      "cat.$.dog",
      "thing1.$.thing2",
      "element.$.localId",
      'element.$["fancy-hyphenated-id"]',
    ]
    for line in lines:
      self.ShouldFailPolymerLocalIdCheck(line)

  def testPolymerLocalIdPasses(self):
    lines = [
      "this.$.id",
      "this.$.localId",
      'this.$["fancy-id"]',
      "this.page.$.flushForTesting()",
    ]
    for line in lines:
      self.ShouldPassPolymerLocalIdCheck(line)

  def ShouldFailVariableNameCheck(self, line):
    """Checks that var unix_hacker, $dollar are style errors."""
    error = self.checker.VariableNameCheck(1, line)
    self.assertNotEqual("", error, "Should be flagged as style error: " + line)
    highlight = test_util.GetHighlight(line, error)
    self.assertFalse(any(dm in highlight for dm in _DECLARATION_METHODS))

  def ShouldPassVariableNameCheck(self, line):
    """Checks that variableNamesLikeThis aren"t style errors."""
    self.assertEqual(
      "",
      self.checker.VariableNameCheck(1, line),
      msg="Should not be flagged as style error: " + line,
    )

  def testVariableNameFails(self):
    lines = [
      "%s private_;",
      "%s hostName_ = 'https://google.com';",
      " %s _super_private",
      "  %s unix_hacker = someFunc();",
    ]
    for line in lines:
      for declaration_method in _DECLARATION_METHODS:
        self.ShouldFailVariableNameCheck(line % declaration_method)

  def testVariableNamePasses(self):
    lines = [
      "  %s namesLikeThis = [];",
      " for (%s i = 0; i < 10; ++i) { ",
      "for (%s i in obj) {",
      " %s one, two, three;",
      "  %s magnumPI = {};",
      " %s g_browser = 'da browzer';",
      "/** @const */ %s Bla = options.Bla;",  # goog.scope() replacement.
      " %s $ = function() {",  # For legacy reasons.
      "  %s StudlyCaps = cr.define('bla')",  # Classes.
      " %s SCARE_SMALL_CHILDREN = [",  # TODO(dbeam): add @const in
      # front of all these vars like
      # "/** @const */ %s CONST_VAR = 1;",          # this line has (<--).
    ]
    for line in lines:
      for declaration_method in _DECLARATION_METHODS:
        self.ShouldPassVariableNameCheck(line % declaration_method)


if __name__ == "__main__":
  unittest.main()
