Merge pull request #134 from xsnippet/tests

[Tests]  Update misc tests

Author: lotrien

src/misc/download.js

@@ -1,6 +1,6 @@
 import { getCurrentMode } from './modes'
 
-function download(text, name, mime) {
+const download = (text, name, mime) => {
   // It seems it's the only way to initiate file downloading from JavaScript
   // as of Jan 7, 2018. If you read this and know a better way, please submit
   // a pull request! ;)
@@ -16,7 +16,7 @@ function download(text, name, mime) {
   document.body.removeChild(element)
 }
 
-export function downloadSnippet(snippet) {
+export const downloadSnippet = snippet => {
   // Despite using AceEditor's modes as syntaxes, we can imagine other setup
   // when more or even other syntaxes can be used on API side. Hence, we better
   // be prepared and fallback to "Text" mode if unknown syntaxes it is.

tests/helpers/index.test.js

@@ -1,30 +1,52 @@
+import { fromJS } from 'immutable'
 import { getApiUri } from '../../src/misc/url'
-import { formatDate } from '../../src/misc/snippet'
+import { formatDate, getSnippetTitle } from '../../src/misc/snippet'
 
 describe('misc', () => {
-  it('should return properly formated date', () => {
-    const incomeDate = '2018-03-11T15:28:19'
-
-    expect(formatDate(incomeDate)).toEqual('11.03.2018')
+  describe('snippet related', () => {
+    const snippet = {
+      content: 'const memoizedSyntaxes = useMemo()',
+      created_at: '2019-09-29T19:23:31',
+      id: 364066,
+      syntax: 'javascript',
+      tags: ['trg', 'rtyger'],
+      title: 'some misterious title',
+      updated_at: '2020-01-01T23:28:19',
+    }
+
+    it('should return properly formated date', () => {
+      expect(formatDate(snippet.created_at)).toEqual('29.09.2019')
+    })
+
+    it('should return properly formatted date for Jan', () => {
+      expect(formatDate(snippet.updated_at)).toEqual('01.01.2020')
+    })
+
+    it('should return propper title', () => {
+      expect(getSnippetTitle(fromJS(snippet))).toEqual(snippet.title)
+    })
+
+    it('should return propper title', () => {
+      const untitled = { ...snippet, title: undefined }
+      const expected = `#${untitled.id}, Untitled`
+
+      expect(getSnippetTitle(fromJS(untitled))).toEqual(expected)
+    })
   })
 
-  it('should return properly formatted date for Jan', () => {
-    const incomeDate = '2018-01-01T23:28:19'
-
-    expect(formatDate(incomeDate)).toEqual('01.01.2018')
-  })
+  describe('getApiUri', () => {
+    it('should construct correct url with default api version', () => {
+      process.env.API_BASE_URI = '//api.xsnippet.org'
+      const f = getApiUri('snippets')
 
-  it('should construct correct url with default api version', () => {
-    process.env.API_BASE_URI = '//api.xsnippet.org'
-    const f = getApiUri('snippets')
-
-    expect(f).toBe(`${process.env.API_BASE_URI}/v1/snippets`)
-  })
+      expect(f).toBe(`${process.env.API_BASE_URI}/v1/snippets`)
+    })
 
-  it('should construct correct url with passed api version', () => {
-    process.env.API_BASE_URI = '//api.xsnippet.org'
-    const f = getApiUri('snippets', 'v404')
+    it('should construct correct url with passed api version', () => {
+      process.env.API_BASE_URI = '//api.xsnippet.org'
+      const f = getApiUri('snippets', 'v404')
 
-    expect(f).toBe(`${process.env.API_BASE_URI}/v404/snippets`)
+      expect(f).toBe(`${process.env.API_BASE_URI}/v404/snippets`)
+    })
   })
 })