[Pkg-javascript-commits] [pdf.js] 99/109: Add unit-tests for `removeNullCharacters` (PR 6431 follow-up)

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository pdf.js.

commit 297984e2e9826c599b158890dce113deffc90a01
Author: Jonas Jenwald <jonas.jenwald at gmail.com>
Date:   Mon Sep 14 12:19:29 2015 +0200

    Add unit-tests for `removeNullCharacters` (PR 6431 follow-up)
    
    I overlooked that we already had existing unit-tests for `web/ui_utils.js`, so this PR adds a few tests for `removeNullCharacters` (see PR 6431).
---
 test/unit/ui_utils_spec.js | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/test/unit/ui_utils_spec.js b/test/unit/ui_utils_spec.js
index 757aef2..5a46867 100644
--- a/test/unit/ui_utils_spec.js
+++ b/test/unit/ui_utils_spec.js
@@ -1,10 +1,21 @@
 /* -*- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -*- */
 /* vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */
-/* globals expect, it, describe, binarySearchFirstItem */
+/* globals expect, it, describe, binarySearchFirstItem, removeNullCharacters */
 
 'use strict';
 
 describe('ui_utils', function() {
+  describe('removeNullCharacters', function() {
+    it('should not modify string without null characters', function() {
+      var str = 'string without null chars';
+      expect(removeNullCharacters(str)).toEqual('string without null chars');
+    });
+
+    it('should modify string with null characters', function() {
+      var str = 'string\x00With\x00Null\x00Chars';
+      expect(removeNullCharacters(str)).toEqual('stringWithNullChars');
+    });
+  });
 
   describe('binary search', function() {
     function isTrue(boolean) {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/pdf.js.git