gcg/static-cms
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(core): force multiline flag for editor component patterns (#3082)

Browse Source
This commit is contained in:
Shawn Erquhart 2020-01-14 01:44:37 -05:00 committed by Erez Rokah
parent 09564bf8b6
commit 476f45096e
2 changed files with 39 additions and 38 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
packages/netlify-cms-core/src/valueObjects/EditorComponent.js
View File

@ -25,7 +25,8 @@ export default function createEditorComponent(config) {
type,
icon,
widget,
pattern,
// enforce multiline flag, exclude others
pattern: new RegExp(pattern, 'm'),
fromBlock: bind(fromBlock) || (() => ({})),
toBlock: bind(toBlock) || (() => 'Plugin'),
toPreview: bind(toPreview) || (!widget && (bind(toBlock) || (() => 'Plugin'))),

74
packages/netlify-cms-widget-markdown/src/serializers/__tests__/remarkShortcodes.spec.js
View File

@ -1,4 +1,3 @@
import { stripIndent } from 'common-tags';
import { remarkParseShortcodes } from '../remarkShortcodes';
// Stub of Remark Parser
@ -11,43 +10,44 @@ function process(value, plugins, processEat = () => {}) {
Parser.prototype.blockTokenizers.shortcode(eat, value);
}
function EditorComponent({ id = 'foo', fromBlock = jest.fn(), pattern }) {
// initialize pattern as RegExp as done in the EditorComponent value object
return { id, fromBlock, pattern: new RegExp(pattern, 'm') };
}
describe('remarkParseShortcodes', () => {
let editorComponent;
beforeEach(() => {
editorComponent = {
id: 'foo',
pattern: /bar/,
fromBlock: jest.fn(),
};
describe('pattern matching', () => {
it('should work', () => {
const editorComponent = EditorComponent({ pattern: /bar/ });
process('foo bar', [editorComponent]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['bar']));
});
it('should match value surrounded in newlines', () => {
const editorComponent = EditorComponent({ pattern: /^bar$/ });
process('foo\n\nbar\n', [editorComponent]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['bar']));
});
it('should match multiline shortcodes', () => {
const editorComponent = EditorComponent({ pattern: /^foo\nbar$/ });
process('foo\nbar', [editorComponent]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['foo\nbar']));
});
it('should match multiline shortcodes with empty lines', () => {
const editorComponent = EditorComponent({ pattern: /^foo\n\nbar$/ });
process('foo\n\nbar', [editorComponent]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(
expect.arrayContaining(['foo\n\nbar']),
);
});
});
it('should parse shortcodes', () => {
process('foo bar', [editorComponent]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['bar']));
});
it('should parse multiline shortcodes', () => {
const value = stripIndent`
foo
bar
`;
process(value, [{ ...editorComponent, pattern: /^foo\nbar$/ }]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['foo\nbar']));
});
it('should parse multiline shortcodes with empty lines', () => {
const value = stripIndent`
foo
bar
`;
process(value, [{ ...editorComponent, pattern: /^foo\n\nbar$/ }]);
expect(editorComponent.fromBlock).toHaveBeenCalledWith(expect.arrayContaining(['foo\n\nbar']));
});
it('should produce shortcode node', () => {
const processEat = jest.fn();
const shortcodeData = { bar: 'baz' };
const expectedNode = { type: 'shortcode', data: { shortcode: 'foo', shortcodeData } };
editorComponent.fromBlock = () => shortcodeData;
process('foo bar', [editorComponent], processEat);
expect(processEat).toHaveBeenCalledWith(expectedNode);
describe('output', () => {
it('should be a remark shortcode node', () => {
const processEat = jest.fn();
const shortcodeData = { bar: 'baz' };
const expectedNode = { type: 'shortcode', data: { shortcode: 'foo', shortcodeData } };
const editorComponent = EditorComponent({ pattern: /bar/, fromBlock: () => shortcodeData });
process('foo bar', [editorComponent], processEat);
expect(processEat).toHaveBeenCalledWith(expectedNode);
});
});
});