mirror of
https://github.com/mozilla/pdf.js.git
synced 2025-04-20 23:28:06 +02:00
Removes global PDFJS usage from the src/core/.
This commit is contained in:
parent
21ed8ff71d
commit
bda5e6235e
22 changed files with 376 additions and 299 deletions
50
test/unit/dom_utils_spec.js
Normal file
50
test/unit/dom_utils_spec.js
Normal file
|
@ -0,0 +1,50 @@
|
|||
/* globals expect, it, describe, PDFJS, isExternalLinkTargetSet, LinkTarget,
|
||||
getFilenameFromUrl */
|
||||
|
||||
'use strict';
|
||||
|
||||
describe('dom_utils', function() {
|
||||
describe('getFilenameFromUrl', function() {
|
||||
it('should get the filename from an absolute URL', function() {
|
||||
var url = 'http://server.org/filename.pdf';
|
||||
var result = getFilenameFromUrl(url);
|
||||
var expected = 'filename.pdf';
|
||||
expect(result).toEqual(expected);
|
||||
});
|
||||
|
||||
it('should get the filename from a relative URL', function() {
|
||||
var url = '../../filename.pdf';
|
||||
var result = getFilenameFromUrl(url);
|
||||
var expected = 'filename.pdf';
|
||||
expect(result).toEqual(expected);
|
||||
});
|
||||
});
|
||||
|
||||
describe('isExternalLinkTargetSet', function() {
|
||||
// Save the current state, to avoid interfering with other tests.
|
||||
var previousExternalLinkTarget = PDFJS.externalLinkTarget;
|
||||
|
||||
it('handles the predefined LinkTargets', function() {
|
||||
for (var key in LinkTarget) {
|
||||
var linkTarget = LinkTarget[key];
|
||||
PDFJS.externalLinkTarget = linkTarget;
|
||||
|
||||
expect(isExternalLinkTargetSet()).toEqual(!!linkTarget);
|
||||
}
|
||||
});
|
||||
|
||||
it('handles incorrect LinkTargets', function() {
|
||||
var targets = [true, '', false, -1, '_blank', null];
|
||||
|
||||
for (var i = 0, ii = targets.length; i < ii; i++) {
|
||||
var linkTarget = targets[i];
|
||||
PDFJS.externalLinkTarget = linkTarget;
|
||||
|
||||
expect(isExternalLinkTargetSet()).toEqual(false);
|
||||
}
|
||||
});
|
||||
|
||||
// Reset the state.
|
||||
PDFJS.externalLinkTarget = previousExternalLinkTarget;
|
||||
});
|
||||
});
|
|
@ -37,6 +37,7 @@
|
|||
<script src="cmap_spec.js"></script>
|
||||
<script src="annotation_layer_spec.js"></script>
|
||||
<script src="network_spec.js"></script>
|
||||
<script src="dom_utils_spec.js"></script>
|
||||
|
||||
<script>
|
||||
'use strict';
|
||||
|
@ -50,11 +51,11 @@
|
|||
'pdfjs/core/fonts', 'pdfjs/core/ps_parser', 'pdfjs/core/function',
|
||||
'pdfjs/core/parser', 'pdfjs/core/evaluator', 'pdfjs/core/cmap',
|
||||
'pdfjs/core/worker', 'pdfjs/core/network', 'pdfjs/display/api',
|
||||
'pdfjs/display/metadata'],
|
||||
'pdfjs/display/metadata', 'pdfjs/display/dom_utils'],
|
||||
function (sharedUtil, sharedGlobal, corePrimitives, coreAnnotation,
|
||||
coreCrypto, coreStream, coreFonts, corePsParser, coreFunction,
|
||||
coreParser, coreEvaluator, coreCMap, coreWorker,
|
||||
coreNetwork, displayAPI, displayMetadata) {
|
||||
coreParser, coreEvaluator, coreCMap, coreWorker, coreNetwork,
|
||||
displayAPI, displayMetadata, displayDOMUtils) {
|
||||
|
||||
pdfjsLibs = {
|
||||
sharedUtil: sharedUtil,
|
||||
|
@ -72,7 +73,8 @@
|
|||
coreWorker: coreWorker,
|
||||
coreNetwork: coreNetwork,
|
||||
displayAPI: displayAPI,
|
||||
displayMetadata: displayMetadata
|
||||
displayMetadata: displayMetadata,
|
||||
displayDOMUtils: displayDOMUtils
|
||||
};
|
||||
|
||||
// Expose all loaded internal exported members to global scope.
|
||||
|
|
|
@ -1,26 +1,9 @@
|
|||
/* globals expect, it, describe, combineUrl, Dict, isDict, Name, PDFJS,
|
||||
stringToPDFString, isExternalLinkTargetSet, LinkTarget,
|
||||
removeNullCharacters, getFilenameFromUrl */
|
||||
stringToPDFString, removeNullCharacters */
|
||||
|
||||
'use strict';
|
||||
|
||||
describe('util', function() {
|
||||
describe('getFilenameFromUrl', function() {
|
||||
it('should get the filename from an absolute URL', function() {
|
||||
var url = 'http://server.org/filename.pdf';
|
||||
var result = getFilenameFromUrl(url);
|
||||
var expected = 'filename.pdf';
|
||||
expect(result).toEqual(expected);
|
||||
});
|
||||
|
||||
it('should get the filename from a relative URL', function() {
|
||||
var url = '../../filename.pdf';
|
||||
var result = getFilenameFromUrl(url);
|
||||
var expected = 'filename.pdf';
|
||||
expect(result).toEqual(expected);
|
||||
});
|
||||
});
|
||||
|
||||
describe('combineUrl', function() {
|
||||
it('absolute url with protocol stays as is', function() {
|
||||
var baseUrl = 'http://server/index.html';
|
||||
|
@ -114,34 +97,6 @@ describe('util', function() {
|
|||
});
|
||||
});
|
||||
|
||||
describe('isExternalLinkTargetSet', function() {
|
||||
// Save the current state, to avoid interfering with other tests.
|
||||
var previousExternalLinkTarget = PDFJS.externalLinkTarget;
|
||||
|
||||
it('handles the predefined LinkTargets', function() {
|
||||
for (var key in LinkTarget) {
|
||||
var linkTarget = LinkTarget[key];
|
||||
PDFJS.externalLinkTarget = linkTarget;
|
||||
|
||||
expect(isExternalLinkTargetSet()).toEqual(!!linkTarget);
|
||||
}
|
||||
});
|
||||
|
||||
it('handles incorrect LinkTargets', function() {
|
||||
var targets = [true, '', false, -1, '_blank', null];
|
||||
|
||||
for (var i = 0, ii = targets.length; i < ii; i++) {
|
||||
var linkTarget = targets[i];
|
||||
PDFJS.externalLinkTarget = linkTarget;
|
||||
|
||||
expect(isExternalLinkTargetSet()).toEqual(false);
|
||||
}
|
||||
});
|
||||
|
||||
// Reset the state.
|
||||
PDFJS.externalLinkTarget = previousExternalLinkTarget;
|
||||
});
|
||||
|
||||
describe('removeNullCharacters', function() {
|
||||
it('should not modify string without null characters', function() {
|
||||
var str = 'string without null chars';
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue