BREAKING CHANGE: Make class Term constructors internal (#338)

These constructors take the `id` at face value so can be interpreted as the
wrong term type if an invalid value is given; for instance a `NamedNode` can
be interpred as a `Literal` if instantiated with `new NamedNode('"hellow world"')`.

Consumers of the library should use the `DataFactory` instead to create new terms.

Author: jeswr

src/index.js

@@ -9,15 +9,6 @@ import * as Util from './N3Util';
 import {
   default as DataFactory,
 
-  Term,
-  NamedNode,
-  Literal,
-  BlankNode,
-  Variable,
-  DefaultGraph,
-  Quad,
-  Triple,
-
   termFromId,
   termToId,
 } from './N3DataFactory';
@@ -34,15 +25,6 @@ export {
 
   DataFactory,
 
-  Term,
-  NamedNode,
-  Literal,
-  BlankNode,
-  Variable,
-  DefaultGraph,
-  Quad,
-  Triple,
-
   termFromId,
   termToId,
 };
@@ -59,15 +41,6 @@ export default {
 
   DataFactory,
 
-  Term,
-  NamedNode,
-  Literal,
-  BlankNode,
-  Variable,
-  DefaultGraph,
-  Quad,
-  Triple,
-
   termFromId,
   termToId,
 };

test/BlankNode-test.js

@@ -1,4 +1,4 @@
-import { BlankNode, Term } from '../src/';
+import { BlankNode, Term } from '../src/N3DataFactory';
 
 describe('BlankNode', () => {
   describe('The BlankNode module', () => {

test/DefaultGraph-test.js

@@ -1,4 +1,4 @@
-import { DefaultGraph, Term } from '../src/';
+import { DefaultGraph, Term } from '../src/N3DataFactory';
 
 describe('DefaultGraph', () => {
   describe('The DefaultGraph module', () => {

test/Literal-test.js

@@ -1,4 +1,4 @@
-import { Literal, NamedNode, Term } from '../src/';
+import { Literal, NamedNode, Term } from '../src/N3DataFactory';
 
 describe('Literal', () => {
   describe('The Literal module', () => {

test/N3DataFactory-test.js

@@ -1,12 +1,12 @@
 import {
-  DataFactory,
   NamedNode,
   Literal,
   BlankNode,
   Variable,
   DefaultGraph,
   Quad,
-} from '../src/';
+} from '../src/N3DataFactory';
+import { DataFactory } from '../src';
 
 describe('DataFactory', () => {
   describe('namedNode', () => {

test/N3Parser-test.js

@@ -1,4 +1,5 @@
-import { Parser, NamedNode, BlankNode, Quad, termFromId } from '../src/';
+import { Parser, termFromId } from '../src/';
+import { NamedNode, BlankNode, Quad } from '../src/N3DataFactory';
 
 const BASE_IRI = 'http://example.org/';
 

test/N3Store-test.js

@@ -1,11 +1,13 @@
 import {
   Store,
+  termFromId, termToId,
+} from '../src/';
+import {
   NamedNode,
   Literal,
   DefaultGraph,
   Quad,
-  termFromId, termToId,
-} from '../src/';
+} from '../src/N3DataFactory';
 import namespaces from '../src/IRIs';
 import chai, { expect } from 'chai';
 import { Readable } from 'readable-stream';

test/N3StreamParser-test.js

@@ -1,4 +1,5 @@
-import { StreamParser, NamedNode } from '../src/';
+import { StreamParser } from '../src/';
+import { NamedNode } from '../src/N3DataFactory';
 import { Readable, Writable } from 'readable-stream';
 
 describe('StreamParser', () => {

test/N3StreamWriter-test.js

@@ -1,4 +1,5 @@
-import { StreamWriter, Quad, NamedNode, termFromId } from '../src/';
+import { StreamWriter, termFromId } from '../src/';
+import { Quad, NamedNode } from '../src/N3DataFactory';
 import { Readable, Writable } from 'readable-stream';
 
 describe('StreamWriter', () => {

test/N3Writer-test.js

@@ -1,11 +1,11 @@
 import {
   Writer,
-  NamedNode,
-  BlankNode,
-  Literal,
-  Quad,
   termFromId,
 } from '../src/';
+import { NamedNode,
+  BlankNode,
+  Literal,
+  Quad } from '../src/N3DataFactory';
 import namespaces from '../src/IRIs';
 
 const { xsd } = namespaces;