mirror of https://github.com/Squidex/squidex.git
seamys
8 years ago
28 changed files with 2889 additions and 2790 deletions
@ -0,0 +1,14 @@ |
|||
// ==========================================================================
|
|||
// Squidex Headless CMS
|
|||
// ==========================================================================
|
|||
// Copyright (c) Squidex UG (haftungsbeschraenkt)
|
|||
// All rights reserved. Licensed under the MIT license.
|
|||
// ==========================================================================
|
|||
|
|||
namespace Squidex.Pipeline |
|||
{ |
|||
public sealed class ETagOptions |
|||
{ |
|||
public bool Strong { get; set; } |
|||
} |
|||
} |
|||
@ -1,41 +0,0 @@ |
|||
const { Runner, run } = require('tslint/lib/runner'); |
|||
|
|||
const options = JSON.parse(process.argv[2]) || {}; |
|||
|
|||
function logToParent(message) { |
|||
process.stdout.write(message); |
|||
} |
|||
|
|||
function runLinter(runnerOptions, write) { |
|||
const logTsLint = message => { |
|||
write('tslint:' + message) |
|||
}; |
|||
|
|||
if (run) { |
|||
const logger = { log: logTsLint, error: logTsLint }; |
|||
|
|||
return run(runnerOptions, logger); |
|||
} else if (Runner) { |
|||
return new Promise(resolve => { |
|||
new Runner(runnerOptions, { write: logTsLint }).run(resolve); |
|||
}); |
|||
} else { |
|||
write('tsinfo:Unable to launch tslint. No suitable runner found.'); |
|||
} |
|||
} |
|||
|
|||
logToParent('tsinfo:Linting started in separate process...'); |
|||
|
|||
const runnerOptions = Object.assign({ |
|||
exclude: [], |
|||
format: 'json' |
|||
}, options); |
|||
|
|||
runLinter(runnerOptions, logToParent) |
|||
.then(() => { |
|||
logToParent('tsinfo:Linting complete.'); |
|||
|
|||
process.exit(); |
|||
}).catch(error => { |
|||
logToParent(`tserror:Error starting linter: ${error}\n${error.stack}`); |
|||
}); |
|||
@ -1,157 +0,0 @@ |
|||
const path = require('path'); |
|||
const chalk = require('chalk'); |
|||
const { fork } = require('child_process'); |
|||
|
|||
function apply(options, compiler) { |
|||
let linterProcess; |
|||
let linterPromise; |
|||
let linterIteration = 0; |
|||
|
|||
function compileHook() { |
|||
if (linterProcess && linterProcess.kill) { |
|||
// Exits any outstanding child process if one exists
|
|||
linterProcess.kill(); |
|||
} |
|||
|
|||
let { files = [] } = options; |
|||
|
|||
if (!files.length) { |
|||
process.stdout.write(chalk.yellow.bold('\n[tslint-plugin] No `files` option specified.\n')); |
|||
return; |
|||
} |
|||
|
|||
options.files = Array.isArray(files) ? files : [files]; |
|||
|
|||
// Spawn a child process to run the linter
|
|||
linterProcess = fork(path.resolve(__dirname, 'linter.js'), [JSON.stringify(options)], { |
|||
silent: true |
|||
}); |
|||
|
|||
// Use the iteration to cancel previous promises.
|
|||
linterIteration++; |
|||
|
|||
linterPromise = new Promise(resolve => { |
|||
const linterOutBuffer = []; |
|||
|
|||
linterProcess.stdout.on('data', (message) => { |
|||
if (message) { |
|||
const msg = message.toString(); |
|||
|
|||
for (let line of msg.split('\n')) { |
|||
const indexOfSeparator = line.indexOf(':'); |
|||
|
|||
if (indexOfSeparator > 0) { |
|||
const type = line.substring(0, indexOfSeparator); |
|||
const body = line.substring(indexOfSeparator + 1); |
|||
|
|||
switch (type) { |
|||
case 'tslint': { |
|||
const json = JSON.parse(body); |
|||
|
|||
for (let item of json) { |
|||
linterOutBuffer.push(item); |
|||
} |
|||
|
|||
break; |
|||
} |
|||
case 'tsinfo': { |
|||
process.stdout.write(chalk.cyan(`[tslint-plugin] ${body}\n`)); |
|||
break; |
|||
} |
|||
case 'tserror': { |
|||
process.stderr.write(chalk.red(`[tslint-plugin] ${body}\n`)); |
|||
break; |
|||
} |
|||
default: { |
|||
process.stderr.write(msg); |
|||
} |
|||
} |
|||
} else { |
|||
process.stdout.write(line); |
|||
} |
|||
} |
|||
} |
|||
}); |
|||
|
|||
linterProcess.once('exit', () => { |
|||
resolve({ iteration: linterIteration, out: linterOutBuffer }); |
|||
|
|||
// Clean up the linterProcess when finished
|
|||
delete linterProcess; |
|||
}); |
|||
}); |
|||
} |
|||
|
|||
function createError(message) { |
|||
const error = new Error(message); |
|||
delete error.stackTrace; |
|||
return error; |
|||
} |
|||
|
|||
function emitHook(compilation, callback) { |
|||
if (linterPromise && options.waitForLinting) { |
|||
linterPromise.then(result => { |
|||
for (let r of result.out) { |
|||
const msg = `${r.name}:${r.startPosition.line + 1}:${r.startPosition.character + 1} [tslint] ${r.ruleName}: ${r.failure}`; |
|||
|
|||
if (r.ruleSeverity === 'ERROR' || options.warningsAsError) { |
|||
compilation.errors.push(createError(msg)); |
|||
} else { |
|||
compilation.warnings.push(createError(msg)); |
|||
} |
|||
} |
|||
|
|||
callback(); |
|||
}); |
|||
} else { |
|||
callback(); |
|||
} |
|||
} |
|||
|
|||
function doneHook() { |
|||
const currentIteration = linterIteration; |
|||
|
|||
if (linterPromise && !options.waitForLinting) { |
|||
let isResolved = false; |
|||
|
|||
linterPromise.then(result => { |
|||
isResolved = true; |
|||
|
|||
// If the iterations are not the same another process has already been started and we cancel these results.
|
|||
if (result.iteration === currentIteration) { |
|||
for (let r of result.out) { |
|||
const msg = `${r.name}:${r.startPosition.line + 1}:${r.startPosition.character + 1} [tslint] ${r.ruleName}: ${r.failure}`; |
|||
|
|||
if (r.ruleSeverity === 'ERROR' || options.warningsAsError) { |
|||
process.stderr.write(chalk.red(msg + '\n')); |
|||
} else { |
|||
process.stdout.write(chalk.yellow(msg + '\n')); |
|||
} |
|||
} |
|||
} |
|||
}); |
|||
|
|||
if (!isResolved) { |
|||
process.stdout.write(chalk.cyan(`[tslint-plugin] Waiting for results...\n`)); |
|||
} |
|||
} |
|||
} |
|||
|
|||
if (compiler.hooks) { |
|||
// Webpack 4
|
|||
compiler.hooks.compile.tap('TSLintWebpackPlugin', compileHook); |
|||
compiler.hooks.emit.tapAsync('TSLintWebpackPlugin', emitHook); |
|||
compiler.hooks.done.tap('TSLintWebpackPlugin', doneHook); |
|||
} else { |
|||
// Backwards compatibility
|
|||
compiler.plugin('compile', compileHook); |
|||
compiler.plugin('emit', emitHook); |
|||
compiler.plugin('done', doneHook); |
|||
} |
|||
} |
|||
|
|||
module.exports = function TSLintWebpackPlugin(options = {}) { |
|||
return { |
|||
apply: apply.bind(this, options) |
|||
}; |
|||
}; |
|||
@ -1,16 +1,15 @@ |
|||
const webpack = require('webpack'), |
|||
webpackMerge = require('webpack-merge'), |
|||
path = require('path'), |
|||
helpers = require('./helpers'), |
|||
commonConfig = require('./webpack.config.js'); |
|||
const webpack = require('webpack'), |
|||
webpackMerge = require('webpack-merge'), |
|||
path = require('path'), |
|||
helpers = require('./helpers'), |
|||
commonConfig = require('./webpack.config.js'); |
|||
|
|||
module.exports = webpackMerge(commonConfig, { |
|||
mode: 'development', |
|||
|
|||
/** |
|||
* Source map for Karma from the help of karma-sourcemap-loader & karma-webpack |
|||
* Source map for Karma from the help of karma-sourcemap-loader & karma-webpack. |
|||
* |
|||
* Do not change, leave as is or it wont work. |
|||
* See: https://webpack.js.org/configuration/devtool/
|
|||
*/ |
|||
devtool: 'inline-source-map' |
|||
|
|||
File diff suppressed because it is too large
@ -0,0 +1,88 @@ |
|||
// ==========================================================================
|
|||
// Squidex Headless CMS
|
|||
// ==========================================================================
|
|||
// Copyright (c) Squidex UG (haftungsbeschraenkt)
|
|||
// All rights reserved. Licensed under the MIT license.
|
|||
// ==========================================================================
|
|||
|
|||
using System.Collections.Generic; |
|||
using System.Threading.Tasks; |
|||
using FakeItEasy; |
|||
using Microsoft.AspNetCore.Http; |
|||
using Microsoft.AspNetCore.Mvc; |
|||
using Microsoft.AspNetCore.Mvc.Abstractions; |
|||
using Microsoft.AspNetCore.Mvc.Filters; |
|||
using Microsoft.AspNetCore.Routing; |
|||
using Microsoft.Extensions.Options; |
|||
using Microsoft.Net.Http.Headers; |
|||
using Xunit; |
|||
|
|||
namespace Squidex.Pipeline |
|||
{ |
|||
public class ETagFilterTests |
|||
{ |
|||
private readonly HttpContext httpContext = new DefaultHttpContext(); |
|||
private readonly ActionExecutingContext executingContext; |
|||
private readonly ActionExecutedContext executedContext; |
|||
private readonly ETagFilter sut = new ETagFilter(Options.Create(new ETagOptions())); |
|||
|
|||
public ETagFilterTests() |
|||
{ |
|||
var actionContext = new ActionContext(httpContext, new RouteData(), new ActionDescriptor()); |
|||
|
|||
var filters = new List<IFilterMetadata>(); |
|||
|
|||
executingContext = new ActionExecutingContext(actionContext, filters, new Dictionary<string, object>(), this); |
|||
executedContext = new ActionExecutedContext(actionContext, filters, this) |
|||
{ |
|||
Result = new OkResult() |
|||
}; |
|||
} |
|||
|
|||
[Fact] |
|||
public async Task Should_convert_strong_to_weak_tag() |
|||
{ |
|||
httpContext.Response.Headers[HeaderNames.ETag] = "13"; |
|||
|
|||
await sut.OnActionExecutionAsync(executingContext, () => Task.FromResult(executedContext)); |
|||
|
|||
Assert.Equal("W/13", httpContext.Response.Headers[HeaderNames.ETag]); |
|||
} |
|||
|
|||
[Fact] |
|||
public async Task Should_not_convert_empty_strong_to_weak_tag() |
|||
{ |
|||
httpContext.Response.Headers[HeaderNames.ETag] = string.Empty; |
|||
|
|||
await sut.OnActionExecutionAsync(executingContext, () => Task.FromResult(executedContext)); |
|||
|
|||
Assert.Null((string)httpContext.Response.Headers[HeaderNames.ETag]); |
|||
} |
|||
|
|||
[Fact] |
|||
public async Task Should_return_304_for_same_etags() |
|||
{ |
|||
httpContext.Request.Method = HttpMethods.Get; |
|||
httpContext.Request.Headers[HeaderNames.IfNoneMatch] = "W/13"; |
|||
|
|||
httpContext.Response.Headers[HeaderNames.ETag] = "13"; |
|||
|
|||
await sut.OnActionExecutionAsync(executingContext, () => Task.FromResult(executedContext)); |
|||
|
|||
Assert.Equal(304, (executedContext.Result as StatusCodeResult).StatusCode); |
|||
} |
|||
|
|||
[Fact] |
|||
public async Task Should_not_return_304_for_different_etags() |
|||
{ |
|||
httpContext.Request.Method = HttpMethods.Get; |
|||
httpContext.Request.Headers[HeaderNames.IfNoneMatch] = "W/11"; |
|||
|
|||
httpContext.Response.Headers[HeaderNames.ETag] = "13"; |
|||
|
|||
await sut.OnActionExecutionAsync(executingContext, () => Task.FromResult(executedContext)); |
|||
|
|||
Assert.Equal(200, (executedContext.Result as StatusCodeResult).StatusCode); |
|||
} |
|||
} |
|||
} |
|||
Loading…
Reference in new issue