Skip to content

Commit

Permalink
[folder structure] installedPlugins -> plugins
Browse files Browse the repository at this point in the history
  • Loading branch information
jbudz committed Jul 5, 2016
1 parent 5588983 commit 28f30f8
Show file tree
Hide file tree
Showing 17 changed files with 24 additions and 24 deletions.
2 changes: 1 addition & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ target
/esvm
.htpasswd
.eslintcache
installedPlugins
plugins
disabledPlugins
webpackstats.json
config/kibana.dev.yml
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ for development and fixing individual tests.
This should work super if you're using the [Kibana plugin generator](https://github.com/elastic/generator-kibana-plugin). If you're not using the generator, well, you're on your own. We suggest you look at how the generator works.

`npm run test:dev -- --kbnServer.testsBundle.pluginId=some_special_plugin --kbnServer.plugin-path=../some_special_plugin`
Run the tests for just your particular plugin. Assuming you plugin lives outside of the `installedPlugins directory`, which it should.
Run the tests for just your particular plugin. Assuming you plugin lives outside of the `plugins directory`, which it should.

#### Running browser automation tests:

Expand Down
File renamed without changes.
Empty file added plugins/.empty
Empty file.
2 changes: 1 addition & 1 deletion src/cli/serve/serve.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ module.exports = function (program) {
'times to specify multiple directories',
pluginDirCollector,
[
fromRoot('installedPlugins'),
fromRoot('plugins'),
fromRoot('src/plugins')
]
)
Expand Down
6 changes: 3 additions & 3 deletions src/cli_plugin/install/__tests__/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ describe('kibana cli', function () {
let options = {};
const kbnPackage = { version: 1234 };
beforeEach(function () {
options = { pluginDir: fromRoot('installedPlugins') };
options = { pluginDir: fromRoot('plugins') };
});

describe('timeout option', function () {
Expand Down Expand Up @@ -143,10 +143,10 @@ describe('kibana cli', function () {

describe('pluginDir option', function () {

it('should default to installedPlugins', function () {
it('should default to plugins', function () {
const settings = parse(command, options, kbnPackage);

expect(settings.pluginDir).to.be(fromRoot('installedPlugins'));
expect(settings.pluginDir).to.be(fromRoot('plugins'));
});

it('should set settings.config property', function () {
Expand Down
2 changes: 1 addition & 1 deletion src/cli_plugin/install/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default function pluginInstall(program) {
.option(
'-d, --plugin-dir <path>',
'path to the directory where plugins are stored',
fromRoot('installedPlugins')
fromRoot('plugins')
)
.description('install a plugin',
`Common examples:
Expand Down
6 changes: 3 additions & 3 deletions src/cli_plugin/list/__tests__/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,15 @@ describe('kibana cli', function () {
let command;
const options = {};
beforeEach(function () {
command = { pluginDir: fromRoot('installedPlugins') };
command = { pluginDir: fromRoot('plugins') };
});

describe('pluginDir option', function () {

it('should default to installedPlugins', function () {
it('should default to plugins', function () {
const settings = parse(command, options);

expect(settings.pluginDir).to.be(fromRoot('installedPlugins'));
expect(settings.pluginDir).to.be(fromRoot('plugins'));
});

it('should set settings.config property', function () {
Expand Down
2 changes: 1 addition & 1 deletion src/cli_plugin/list/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ export default function pluginList(program) {
.option(
'-d, --plugin-dir <path>',
'path to the directory where plugins are stored',
fromRoot('installedPlugins')
fromRoot('plugins')
)
.description('list installed plugins')
.action(processCommand);
Expand Down
6 changes: 3 additions & 3 deletions src/cli_plugin/remove/__tests__/settings.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ describe('kibana cli', function () {
let options = {};
const kbnPackage = { version: 1234 };
beforeEach(function () {
options = { pluginDir: fromRoot('installedPlugins') };
options = { pluginDir: fromRoot('plugins') };
});

describe('quiet option', function () {
Expand Down Expand Up @@ -72,10 +72,10 @@ describe('kibana cli', function () {

describe('pluginDir option', function () {

it('should default to installedPlugins', function () {
it('should default to plugins', function () {
const settings = parse(command, options, kbnPackage);

expect(settings.pluginDir).to.be(fromRoot('installedPlugins'));
expect(settings.pluginDir).to.be(fromRoot('plugins'));
});

it('should set settings.config property', function () {
Expand Down
2 changes: 1 addition & 1 deletion src/cli_plugin/remove/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ export default function pluginRemove(program) {
.option(
'-d, --plugin-dir <path>',
'path to the directory where plugins are stored',
fromRoot('installedPlugins')
fromRoot('plugins')
)
.description('remove a plugin',
`common examples:
Expand Down
2 changes: 1 addition & 1 deletion src/server/path/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const CONFIG_PATHS = [

const DATA_PATHS = [
process.env.DATA_PATH,
fromRoot('installedPlugins/.data'),
fromRoot('plugins/.data'),
'/var/lib/kibana'
].filter(Boolean);

Expand Down
2 changes: 1 addition & 1 deletion tasks/build/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ module.exports = function (grunt) {
'copy:devSource',
'babel:build',
'_build:babelOptions',
'_build:installedPlugins',
'_build:plugins',
'_build:packageJson',
'_build:readme',
'_build:babelCache',
Expand Down
6 changes: 3 additions & 3 deletions tasks/build/installed_plugins.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module.exports = function (grunt) {
grunt.registerTask('_build:installedPlugins', function () {
grunt.file.mkdir('build/kibana/installedPlugins');
grunt.file.mkdir('build/kibana/installedPlugins/.data');
grunt.registerTask('_build:plugins', function () {
grunt.file.mkdir('build/kibana/plugins');
grunt.file.mkdir('build/kibana/plugins/.data');
});
};
4 changes: 2 additions & 2 deletions tasks/build/os_packages.js
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ export default (grunt) => {
//config folder is moved to path.conf, exclude {path.home}/config
//uses relative path to --prefix, strip the leading /
'--exclude', `${packages.path.home.slice(1)}/config`,
'--exclude', `${packages.path.home.slice(1)}/installedPlugins/.data`
'--exclude', `${packages.path.home.slice(1)}/plugins/.data`
];
const debOptions = [
'-t', 'deb',
Expand All @@ -60,7 +60,7 @@ export default (grunt) => {
const args = [
`${buildDir}/=${packages.path.home}/`,
`${buildDir}/config/=${packages.path.conf}/`,
`${buildDir}/installedPlugins/.data/=${packages.path.data}/`,
`${buildDir}/plugins/.data/=${packages.path.data}/`,
`${servicesByName.sysv.outputDir}/etc/=/etc/`,
`${servicesByName.systemd.outputDir}/lib/=/lib/`
];
Expand Down
2 changes: 1 addition & 1 deletion tasks/config/packages.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export default (grunt) => {
const FOLDER_HOME = '/usr/share/kibana';
const FOLDER_DATA = '/var/lib/kibana';
const FOLDER_LOGS = '/var/log/kibana';
const FOLDER_PLUGINS = `${FOLDER_HOME}/installedPlugins`;
const FOLDER_PLUGINS = `${FOLDER_HOME}/plugins`;

const FILE_KIBANA_CONF = `${FOLDER_CONFIG}/kibana.yml`;
const FILE_KIBANA_BINARY = `${FOLDER_HOME}/bin/kibana`;
Expand Down
2 changes: 1 addition & 1 deletion tasks/reject_rej_files.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ export default grunt => {
grunt.registerTask('rejectRejFiles', 'Reject any git-apply .rej files', () => {
const ignoredTopLevelDirs = [
'esvm',
'installedPlugins',
'plugins',
'node_modules',
'optimize'
];
Expand Down

0 comments on commit 28f30f8

Please sign in to comment.