Merge pull request #60 from TomSeldon/21/markdown-converter

[tests] Add tests for markdown converter

app/components/markdown-converter/markdown-converter.service.js

@@ -3,7 +3,9 @@
 (function() {
   'use strict';
 
-  angular.module('word-to-markdown.markdown-converter', [])
+  angular.module('word-to-markdown.markdown-converter', [
+    'word-to-markdown.get-content'
+  ])
     .service('markdownConverter', MarkdownConverterService);
 
   /**
@@ -43,7 +45,7 @@
 
   /**
    * @param {string} html - HTML representation of Word document text
-   * @returns {string} Markdown representation of HTML
+   * @returns {Promise.<string>} Markdown representation of HTML
    */
   MarkdownConverterService.prototype.convertFromHtml = function(html) {
     var deferred = this._$q.defer();

app/components/markdown-converter/markdown-converter.service.test.js

@@ -0,0 +1,20 @@
+describe('platform service', function() {
+  /** @type {MarkdownConverterService} */
+  var markdownConverter;
+
+  beforeEach(module('word-to-markdown.markdown-converter'));
+
+  beforeEach(inject(function($injector) {
+    markdownConverter = $injector.get('markdownConverter');
+  }));
+
+  it('expose a method for converting HTML to markdown', function() {
+    expect(markdownConverter.convertFromHtml).toEqual(jasmine.any(Function));
+  });
+
+  it('should expose a method for converting the selected text to markdown', function() {
+    expect(markdownConverter.convertSelectedText).toEqual(jasmine.any(Function));
+  });
+
+  // todo: Add more meaningful tests once suitable mocks for Office and upndown are in place
+});

app/components/office/office.factory.js

@@ -9,6 +9,6 @@
    * @returns {Office|undefined} The Office JS API
    */
   function officeFactory($window) {
-    return $window.Office;
+    return $window.Office || null;
   }
 })();

app/components/platform/platform.service.js

@@ -31,7 +31,7 @@
    * @returns {boolean} Returns true if we're running in a Microsoft Office environment
    */
   PlatformService.prototype.isRunningInOffice = function() {
-    var hasContext = typeof this._office.context !== 'undefined';
+    var hasContext = this._office && typeof this._office.context !== 'undefined';
 
     return hasContext && typeof this._office.context.document !== 'undefined';
   };

bower.json

@@ -14,6 +14,7 @@
   },
   "devDependencies": {
     "angular-mocks": "~1.4.7",
-    "to-markdown": "~1.3.0"
+    "to-markdown": "~1.3.0",
+    "es5-shim": "~4.1.15"
   }
 }

karma.conf.js

@@ -2,7 +2,10 @@ module.exports = function(config) {
   config.set({
     files: [
       // Vendor
+      'bower_components/es5-shim/es5-shim.js',
+      'bower_components/promise-polyfill/Promise.js',
       'bower_components/jquery/jquery.js',
+      'node_modules/upndown/lib/upndown.bundle.js',
       'bower_components/office-ui-fabric/dist/js/jquery.fabric.min.js',
       'bower_components/angular/angular.js',
       'bower_components/angular-mocks/angular-mocks.js',