Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(110)

Unified Diff: third_party/pkg/angular/scripts/git/validate-commit-msg.spec.js

Issue 1058283006: Update pubspecs and dependencies to get pkgbuild tests working. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/pkg/angular/scripts/git/validate-commit-msg.spec.js
diff --git a/third_party/pkg/angular/scripts/git/validate-commit-msg.spec.js b/third_party/pkg/angular/scripts/git/validate-commit-msg.spec.js
deleted file mode 100644
index cc4b4c3c5bacfd05cea8b9cc716fba6383f184b6..0000000000000000000000000000000000000000
--- a/third_party/pkg/angular/scripts/git/validate-commit-msg.spec.js
+++ /dev/null
@@ -1,77 +0,0 @@
-describe('validate-commit-msg.js', function() {
- var m = require('./validate-commit-msg');
- var errors = [];
- var logs = [];
-
- var VALID = true;
- var INVALID = false;
-
- beforeEach(function() {
- errors.length = 0;
- logs.length = 0;
-
- spyOn(console, 'error').andCallFake(function(msg) {
- errors.push(msg.replace(/\x1B\[\d+m/g, '')); // uncolor
- });
-
- spyOn(console, 'log').andCallFake(function(msg) {
- logs.push(msg.replace(/\x1B\[\d+m/g, '')); // uncolor
- });
- });
-
- describe('validateMessage', function() {
-
- it('should be valid', function() {
- expect(m.validateMessage('fixup! fix($compile): something')).toBe(VALID);
- expect(m.validateMessage('fix($compile): something')).toBe(VALID);
- expect(m.validateMessage('feat($location): something')).toBe(VALID);
- expect(m.validateMessage('docs($filter): something')).toBe(VALID);
- expect(m.validateMessage('style($http): something')).toBe(VALID);
- expect(m.validateMessage('refactor($httpBackend): something')).toBe(VALID);
- expect(m.validateMessage('test($resource): something')).toBe(VALID);
- expect(m.validateMessage('chore($controller): something')).toBe(VALID);
- expect(m.validateMessage('chore(foo-bar): something')).toBe(VALID);
- expect(m.validateMessage('chore(*): something')).toBe(VALID);
- expect(m.validateMessage('chore(guide/location): something')).toBe(VALID);
- expect(m.validateMessage('revert(foo): something')).toBe(VALID);
- expect(errors).toEqual([]);
- });
-
-
- it('should validate 100 characters length', function() {
- var msg = "fix($compile): something super mega extra giga tera long, maybe even longer and longer and longer... ";
-
- expect(m.validateMessage(msg)).toBe(INVALID);
- expect(errors).toEqual(['INVALID COMMIT MSG: is longer than 100 characters !']);
- });
-
-
- it('should validate "<type>(<scope>): <subject>" format', function() {
- var msg = 'not correct format';
-
- expect(m.validateMessage(msg)).toBe(INVALID);
- expect(errors).toEqual(['INVALID COMMIT MSG: does not match "<type>(<scope>): <subject>" ! was: not correct format']);
- });
-
-
- it('should validate type', function() {
- expect(m.validateMessage('weird($filter): something')).toBe(INVALID);
- expect(errors).toEqual(['INVALID COMMIT MSG: "weird" is not allowed type !']);
- });
-
-
- it('should allow empty scope', function() {
- expect(m.validateMessage('fix: blablabla')).toBe(VALID);
- });
-
-
- it('should allow dot in scope', function() {
- expect(m.validateMessage('chore(mocks.$httpBackend): something')).toBe(VALID);
- });
-
-
- it('should ignore msg prefixed with "WIP: "', function() {
- expect(m.validateMessage('WIP: bullshit')).toBe(VALID);
- });
- });
-});
« no previous file with comments | « third_party/pkg/angular/scripts/git/validate-commit-msg.js ('k') | third_party/pkg/angular/scripts/karma/snapshot.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698