detects redundant imports that can be eliminated. for #117772 : In order to facilitate review and modification, split the checking code and removing redundant imports code into two PR.
1055 lines
42 KiB
Rust
1055 lines
42 KiB
Rust
use std::cell::Cell;
|
|
use std::fs::File;
|
|
use std::io::{Error, ErrorKind, Read};
|
|
use std::path::{Path, PathBuf};
|
|
use std::{env, fs};
|
|
|
|
use thiserror::Error;
|
|
|
|
use crate::config::config_type::ConfigType;
|
|
#[allow(unreachable_pub)]
|
|
pub use crate::config::file_lines::{FileLines, FileName, Range};
|
|
#[allow(unreachable_pub)]
|
|
pub use crate::config::lists::*;
|
|
#[allow(unreachable_pub)]
|
|
pub use crate::config::macro_names::{MacroSelector, MacroSelectors};
|
|
#[allow(unreachable_pub)]
|
|
pub use crate::config::options::*;
|
|
|
|
#[macro_use]
|
|
pub(crate) mod config_type;
|
|
#[macro_use]
|
|
#[allow(unreachable_pub)]
|
|
pub(crate) mod options;
|
|
|
|
pub(crate) mod file_lines;
|
|
#[allow(unreachable_pub)]
|
|
pub(crate) mod lists;
|
|
pub(crate) mod macro_names;
|
|
|
|
// This macro defines configuration options used in rustfmt. Each option
|
|
// is defined as follows:
|
|
//
|
|
// `name: value type, default value, is stable, description;`
|
|
create_config! {
|
|
// Fundamental stuff
|
|
max_width: usize, 100, true, "Maximum width of each line";
|
|
hard_tabs: bool, false, true, "Use tab characters for indentation, spaces for alignment";
|
|
tab_spaces: usize, 4, true, "Number of spaces per tab";
|
|
newline_style: NewlineStyle, NewlineStyle::Auto, true, "Unix or Windows line endings";
|
|
indent_style: IndentStyle, IndentStyle::Block, false, "How do we indent expressions or items";
|
|
|
|
// Width Heuristics
|
|
use_small_heuristics: Heuristics, Heuristics::Default, true, "Whether to use different \
|
|
formatting for items and expressions if they satisfy a heuristic notion of 'small'";
|
|
width_heuristics: WidthHeuristics, WidthHeuristics::scaled(100), false,
|
|
"'small' heuristic values";
|
|
fn_call_width: usize, 60, true, "Maximum width of the args of a function call before \
|
|
falling back to vertical formatting.";
|
|
attr_fn_like_width: usize, 70, true, "Maximum width of the args of a function-like \
|
|
attributes before falling back to vertical formatting.";
|
|
struct_lit_width: usize, 18, true, "Maximum width in the body of a struct lit before \
|
|
falling back to vertical formatting.";
|
|
struct_variant_width: usize, 35, true, "Maximum width in the body of a struct variant before \
|
|
falling back to vertical formatting.";
|
|
array_width: usize, 60, true, "Maximum width of an array literal before falling \
|
|
back to vertical formatting.";
|
|
chain_width: usize, 60, true, "Maximum length of a chain to fit on a single line.";
|
|
single_line_if_else_max_width: usize, 50, true, "Maximum line length for single line if-else \
|
|
expressions. A value of zero means always break if-else expressions.";
|
|
single_line_let_else_max_width: usize, 50, true, "Maximum line length for single line \
|
|
let-else statements. A value of zero means always format the divergent `else` block \
|
|
over multiple lines.";
|
|
|
|
// Comments. macros, and strings
|
|
wrap_comments: bool, false, false, "Break comments to fit on the line";
|
|
format_code_in_doc_comments: bool, false, false, "Format the code snippet in doc comments.";
|
|
doc_comment_code_block_width: usize, 100, false, "Maximum width for code snippets in doc \
|
|
comments. No effect unless format_code_in_doc_comments = true";
|
|
comment_width: usize, 80, false,
|
|
"Maximum length of comments. No effect unless wrap_comments = true";
|
|
normalize_comments: bool, false, false, "Convert /* */ comments to // comments where possible";
|
|
normalize_doc_attributes: bool, false, false, "Normalize doc attributes as doc comments";
|
|
format_strings: bool, false, false, "Format string literals where necessary";
|
|
format_macro_matchers: bool, false, false,
|
|
"Format the metavariable matching patterns in macros";
|
|
format_macro_bodies: bool, true, false, "Format the bodies of macros";
|
|
skip_macro_invocations: MacroSelectors, MacroSelectors::default(), false,
|
|
"Skip formatting the bodies of macros invoked with the following names.";
|
|
hex_literal_case: HexLiteralCase, HexLiteralCase::Preserve, false,
|
|
"Format hexadecimal integer literals";
|
|
|
|
// Single line expressions and items
|
|
empty_item_single_line: bool, true, false,
|
|
"Put empty-body functions and impls on a single line";
|
|
struct_lit_single_line: bool, true, false,
|
|
"Put small struct literals on a single line";
|
|
fn_single_line: bool, false, false, "Put single-expression functions on a single line";
|
|
where_single_line: bool, false, false, "Force where-clauses to be on a single line";
|
|
|
|
// Imports
|
|
imports_indent: IndentStyle, IndentStyle::Block, false, "Indent of imports";
|
|
imports_layout: ListTactic, ListTactic::Mixed, false, "Item layout inside a import block";
|
|
imports_granularity: ImportGranularity, ImportGranularity::Preserve, false,
|
|
"Merge or split imports to the provided granularity";
|
|
group_imports: GroupImportsTactic, GroupImportsTactic::Preserve, false,
|
|
"Controls the strategy for how imports are grouped together";
|
|
merge_imports: bool, false, false, "(deprecated: use imports_granularity instead)";
|
|
|
|
// Ordering
|
|
reorder_imports: bool, true, true, "Reorder import and extern crate statements alphabetically";
|
|
reorder_modules: bool, true, true, "Reorder module statements alphabetically in group";
|
|
reorder_impl_items: bool, false, false, "Reorder impl items";
|
|
|
|
// Spaces around punctuation
|
|
type_punctuation_density: TypeDensity, TypeDensity::Wide, false,
|
|
"Determines if '+' or '=' are wrapped in spaces in the punctuation of types";
|
|
space_before_colon: bool, false, false, "Leave a space before the colon";
|
|
space_after_colon: bool, true, false, "Leave a space after the colon";
|
|
spaces_around_ranges: bool, false, false, "Put spaces around the .. and ..= range operators";
|
|
binop_separator: SeparatorPlace, SeparatorPlace::Front, false,
|
|
"Where to put a binary operator when a binary expression goes multiline";
|
|
|
|
// Misc.
|
|
remove_nested_parens: bool, true, true, "Remove nested parens";
|
|
combine_control_expr: bool, true, false, "Combine control expressions with function calls";
|
|
short_array_element_width_threshold: usize, 10, true,
|
|
"Width threshold for an array element to be considered short";
|
|
overflow_delimited_expr: bool, false, false,
|
|
"Allow trailing bracket/brace delimited expressions to overflow";
|
|
struct_field_align_threshold: usize, 0, false,
|
|
"Align struct fields if their diffs fits within threshold";
|
|
enum_discrim_align_threshold: usize, 0, false,
|
|
"Align enum variants discrims, if their diffs fit within threshold";
|
|
match_arm_blocks: bool, true, false, "Wrap the body of arms in blocks when it does not fit on \
|
|
the same line with the pattern of arms";
|
|
match_arm_leading_pipes: MatchArmLeadingPipe, MatchArmLeadingPipe::Never, true,
|
|
"Determines whether leading pipes are emitted on match arms";
|
|
force_multiline_blocks: bool, false, false,
|
|
"Force multiline closure bodies and match arms to be wrapped in a block";
|
|
fn_args_layout: Density, Density::Tall, true,
|
|
"(deprecated: use fn_params_layout instead)";
|
|
fn_params_layout: Density, Density::Tall, true,
|
|
"Control the layout of parameters in function signatures.";
|
|
brace_style: BraceStyle, BraceStyle::SameLineWhere, false, "Brace style for items";
|
|
control_brace_style: ControlBraceStyle, ControlBraceStyle::AlwaysSameLine, false,
|
|
"Brace style for control flow constructs";
|
|
trailing_semicolon: bool, true, false,
|
|
"Add trailing semicolon after break, continue and return";
|
|
trailing_comma: SeparatorTactic, SeparatorTactic::Vertical, false,
|
|
"How to handle trailing commas for lists";
|
|
match_block_trailing_comma: bool, false, true,
|
|
"Put a trailing comma after a block based match arm (non-block arms are not affected)";
|
|
blank_lines_upper_bound: usize, 1, false,
|
|
"Maximum number of blank lines which can be put between items";
|
|
blank_lines_lower_bound: usize, 0, false,
|
|
"Minimum number of blank lines which must be put between items";
|
|
edition: Edition, Edition::Edition2015, true, "The edition of the parser (RFC 2052)";
|
|
version: Version, Version::One, false, "Version of formatting rules";
|
|
inline_attribute_width: usize, 0, false,
|
|
"Write an item and its attribute on the same line \
|
|
if their combined width is below a threshold";
|
|
format_generated_files: bool, true, false, "Format generated files";
|
|
|
|
// Options that can change the source code beyond whitespace/blocks (somewhat linty things)
|
|
merge_derives: bool, true, true, "Merge multiple `#[derive(...)]` into a single one";
|
|
use_try_shorthand: bool, false, true, "Replace uses of the try! macro by the ? shorthand";
|
|
use_field_init_shorthand: bool, false, true, "Use field initialization shorthand if possible";
|
|
force_explicit_abi: bool, true, true, "Always print the abi for extern items";
|
|
condense_wildcard_suffixes: bool, false, false, "Replace strings of _ wildcards by a single .. \
|
|
in tuple patterns";
|
|
|
|
// Control options (changes the operation of rustfmt, rather than the formatting)
|
|
color: Color, Color::Auto, false,
|
|
"What Color option to use when none is supplied: Always, Never, Auto";
|
|
required_version: String, env!("CARGO_PKG_VERSION").to_owned(), false,
|
|
"Require a specific version of rustfmt";
|
|
unstable_features: bool, false, false,
|
|
"Enables unstable features. Only available on nightly channel";
|
|
disable_all_formatting: bool, false, true, "Don't reformat anything";
|
|
skip_children: bool, false, false, "Don't reformat out of line modules";
|
|
hide_parse_errors: bool, false, false, "Hide errors from the parser";
|
|
error_on_line_overflow: bool, false, false, "Error if unable to get all lines within max_width";
|
|
error_on_unformatted: bool, false, false,
|
|
"Error if unable to get comments or string literals within max_width, \
|
|
or they are left with trailing whitespaces";
|
|
ignore: IgnoreList, IgnoreList::default(), false,
|
|
"Skip formatting the specified files and directories";
|
|
|
|
// Not user-facing
|
|
verbose: Verbosity, Verbosity::Normal, false, "How much to information to emit to the user";
|
|
file_lines: FileLines, FileLines::all(), false,
|
|
"Lines to format; this is not supported in rustfmt.toml, and can only be specified \
|
|
via the --file-lines option";
|
|
emit_mode: EmitMode, EmitMode::Files, false,
|
|
"What emit Mode to use when none is supplied";
|
|
make_backup: bool, false, false, "Backup changed files";
|
|
print_misformatted_file_names: bool, false, true,
|
|
"Prints the names of mismatched files that were formatted. Prints the names of \
|
|
files that would be formatted when used with `--check` mode. ";
|
|
}
|
|
|
|
#[derive(Error, Debug)]
|
|
#[error("Could not output config: {0}")]
|
|
pub struct ToTomlError(toml::ser::Error);
|
|
|
|
impl PartialConfig {
|
|
pub fn to_toml(&self) -> Result<String, ToTomlError> {
|
|
// Non-user-facing options can't be specified in TOML
|
|
let mut cloned = self.clone();
|
|
cloned.file_lines = None;
|
|
cloned.verbose = None;
|
|
cloned.width_heuristics = None;
|
|
cloned.print_misformatted_file_names = None;
|
|
cloned.merge_imports = None;
|
|
cloned.fn_args_layout = None;
|
|
|
|
::toml::to_string(&cloned).map_err(ToTomlError)
|
|
}
|
|
}
|
|
|
|
impl Config {
|
|
pub(crate) fn version_meets_requirement(&self) -> bool {
|
|
if self.was_set().required_version() {
|
|
let version = env!("CARGO_PKG_VERSION");
|
|
let required_version = self.required_version();
|
|
if version != required_version {
|
|
println!(
|
|
"Error: rustfmt version ({version}) doesn't match the required version \
|
|
({required_version})"
|
|
);
|
|
return false;
|
|
}
|
|
}
|
|
|
|
true
|
|
}
|
|
|
|
/// Constructs a `Config` from the toml file specified at `file_path`.
|
|
///
|
|
/// This method only looks at the provided path, for a method that
|
|
/// searches parents for a `rustfmt.toml` see `from_resolved_toml_path`.
|
|
///
|
|
/// Returns a `Config` if the config could be read and parsed from
|
|
/// the file, otherwise errors.
|
|
pub(super) fn from_toml_path(file_path: &Path) -> Result<Config, Error> {
|
|
let mut file = File::open(&file_path)?;
|
|
let mut toml = String::new();
|
|
file.read_to_string(&mut toml)?;
|
|
Config::from_toml(&toml, file_path.parent().unwrap())
|
|
.map_err(|err| Error::new(ErrorKind::InvalidData, err))
|
|
}
|
|
|
|
/// Resolves the config for input in `dir`.
|
|
///
|
|
/// Searches for `rustfmt.toml` beginning with `dir`, and
|
|
/// recursively checking parents of `dir` if no config file is found.
|
|
/// If no config file exists in `dir` or in any parent, a
|
|
/// default `Config` will be returned (and the returned path will be empty).
|
|
///
|
|
/// Returns the `Config` to use, and the path of the project file if there was
|
|
/// one.
|
|
pub(super) fn from_resolved_toml_path(dir: &Path) -> Result<(Config, Option<PathBuf>), Error> {
|
|
/// Try to find a project file in the given directory and its parents.
|
|
/// Returns the path of a the nearest project file if one exists,
|
|
/// or `None` if no project file was found.
|
|
fn resolve_project_file(dir: &Path) -> Result<Option<PathBuf>, Error> {
|
|
let mut current = if dir.is_relative() {
|
|
env::current_dir()?.join(dir)
|
|
} else {
|
|
dir.to_path_buf()
|
|
};
|
|
|
|
current = fs::canonicalize(current)?;
|
|
|
|
loop {
|
|
match get_toml_path(¤t) {
|
|
Ok(Some(path)) => return Ok(Some(path)),
|
|
Err(e) => return Err(e),
|
|
_ => (),
|
|
}
|
|
|
|
// If the current directory has no parent, we're done searching.
|
|
if !current.pop() {
|
|
break;
|
|
}
|
|
}
|
|
|
|
// If nothing was found, check in the home directory.
|
|
if let Some(home_dir) = dirs::home_dir() {
|
|
if let Some(path) = get_toml_path(&home_dir)? {
|
|
return Ok(Some(path));
|
|
}
|
|
}
|
|
|
|
// If none was found ther either, check in the user's configuration directory.
|
|
if let Some(mut config_dir) = dirs::config_dir() {
|
|
config_dir.push("rustfmt");
|
|
if let Some(path) = get_toml_path(&config_dir)? {
|
|
return Ok(Some(path));
|
|
}
|
|
}
|
|
|
|
Ok(None)
|
|
}
|
|
|
|
match resolve_project_file(dir)? {
|
|
None => Ok((Config::default(), None)),
|
|
Some(path) => Config::from_toml_path(&path).map(|config| (config, Some(path))),
|
|
}
|
|
}
|
|
|
|
pub(crate) fn from_toml(toml: &str, dir: &Path) -> Result<Config, String> {
|
|
let parsed: ::toml::Value = toml
|
|
.parse()
|
|
.map_err(|e| format!("Could not parse TOML: {}", e))?;
|
|
let mut err = String::new();
|
|
let table = parsed
|
|
.as_table()
|
|
.ok_or_else(|| String::from("Parsed config was not table"))?;
|
|
for key in table.keys() {
|
|
if !Config::is_valid_name(key) {
|
|
let msg = &format!("Warning: Unknown configuration option `{key}`\n");
|
|
err.push_str(msg)
|
|
}
|
|
}
|
|
match parsed.try_into() {
|
|
Ok(parsed_config) => {
|
|
if !err.is_empty() {
|
|
eprint!("{err}");
|
|
}
|
|
Ok(Config::default().fill_from_parsed_config(parsed_config, dir))
|
|
}
|
|
Err(e) => {
|
|
err.push_str("Error: Decoding config file failed:\n");
|
|
err.push_str(format!("{e}\n").as_str());
|
|
err.push_str("Please check your config file.");
|
|
Err(err)
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
/// Loads a config by checking the client-supplied options and if appropriate, the
|
|
/// file system (including searching the file system for overrides).
|
|
pub fn load_config<O: CliOptions>(
|
|
file_path: Option<&Path>,
|
|
options: Option<O>,
|
|
) -> Result<(Config, Option<PathBuf>), Error> {
|
|
let over_ride = match options {
|
|
Some(ref opts) => config_path(opts)?,
|
|
None => None,
|
|
};
|
|
|
|
let result = if let Some(over_ride) = over_ride {
|
|
Config::from_toml_path(over_ride.as_ref()).map(|p| (p, Some(over_ride.to_owned())))
|
|
} else if let Some(file_path) = file_path {
|
|
Config::from_resolved_toml_path(file_path)
|
|
} else {
|
|
Ok((Config::default(), None))
|
|
};
|
|
|
|
result.map(|(mut c, p)| {
|
|
if let Some(options) = options {
|
|
options.apply_to(&mut c);
|
|
}
|
|
(c, p)
|
|
})
|
|
}
|
|
|
|
// Check for the presence of known config file names (`rustfmt.toml, `.rustfmt.toml`) in `dir`
|
|
//
|
|
// Return the path if a config file exists, empty if no file exists, and Error for IO errors
|
|
fn get_toml_path(dir: &Path) -> Result<Option<PathBuf>, Error> {
|
|
const CONFIG_FILE_NAMES: [&str; 2] = [".rustfmt.toml", "rustfmt.toml"];
|
|
for config_file_name in &CONFIG_FILE_NAMES {
|
|
let config_file = dir.join(config_file_name);
|
|
match fs::metadata(&config_file) {
|
|
// Only return if it's a file to handle the unlikely situation of a directory named
|
|
// `rustfmt.toml`.
|
|
Ok(ref md) if md.is_file() => return Ok(Some(config_file)),
|
|
// Return the error if it's something other than `NotFound`; otherwise we didn't
|
|
// find the project file yet, and continue searching.
|
|
Err(e) => {
|
|
if e.kind() != ErrorKind::NotFound {
|
|
let ctx = format!("Failed to get metadata for config file {:?}", &config_file);
|
|
let err = anyhow::Error::new(e).context(ctx);
|
|
return Err(Error::new(ErrorKind::Other, err));
|
|
}
|
|
}
|
|
_ => {}
|
|
}
|
|
}
|
|
Ok(None)
|
|
}
|
|
|
|
fn config_path(options: &dyn CliOptions) -> Result<Option<PathBuf>, Error> {
|
|
let config_path_not_found = |path: &str| -> Result<Option<PathBuf>, Error> {
|
|
Err(Error::new(
|
|
ErrorKind::NotFound,
|
|
format!(
|
|
"Error: unable to find a config file for the given path: `{}`",
|
|
path
|
|
),
|
|
))
|
|
};
|
|
|
|
// Read the config_path and convert to parent dir if a file is provided.
|
|
// If a config file cannot be found from the given path, return error.
|
|
match options.config_path() {
|
|
Some(path) if !path.exists() => config_path_not_found(path.to_str().unwrap()),
|
|
Some(path) if path.is_dir() => {
|
|
let config_file_path = get_toml_path(path)?;
|
|
if config_file_path.is_some() {
|
|
Ok(config_file_path)
|
|
} else {
|
|
config_path_not_found(path.to_str().unwrap())
|
|
}
|
|
}
|
|
path => Ok(path.map(ToOwned::to_owned)),
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod test {
|
|
use super::*;
|
|
use std::str;
|
|
|
|
use crate::config::macro_names::MacroName;
|
|
use rustfmt_config_proc_macro::{nightly_only_test, stable_only_test};
|
|
|
|
#[allow(dead_code)]
|
|
mod mock {
|
|
use super::super::*;
|
|
use rustfmt_config_proc_macro::config_type;
|
|
|
|
#[config_type]
|
|
pub(crate) enum PartiallyUnstableOption {
|
|
V1,
|
|
V2,
|
|
#[unstable_variant]
|
|
V3,
|
|
}
|
|
|
|
create_config! {
|
|
// Options that are used by the generated functions
|
|
max_width: usize, 100, true, "Maximum width of each line";
|
|
required_version: String, env!("CARGO_PKG_VERSION").to_owned(), false,
|
|
"Require a specific version of rustfmt.";
|
|
ignore: IgnoreList, IgnoreList::default(), false,
|
|
"Skip formatting the specified files and directories.";
|
|
verbose: Verbosity, Verbosity::Normal, false,
|
|
"How much to information to emit to the user";
|
|
file_lines: FileLines, FileLines::all(), false,
|
|
"Lines to format; this is not supported in rustfmt.toml, and can only be specified \
|
|
via the --file-lines option";
|
|
|
|
// merge_imports deprecation
|
|
imports_granularity: ImportGranularity, ImportGranularity::Preserve, false,
|
|
"Merge imports";
|
|
merge_imports: bool, false, false, "(deprecated: use imports_granularity instead)";
|
|
|
|
// fn_args_layout renamed to fn_params_layout
|
|
fn_args_layout: Density, Density::Tall, true,
|
|
"(deprecated: use fn_params_layout instead)";
|
|
fn_params_layout: Density, Density::Tall, true,
|
|
"Control the layout of parameters in a function signatures.";
|
|
|
|
// Width Heuristics
|
|
use_small_heuristics: Heuristics, Heuristics::Default, true,
|
|
"Whether to use different formatting for items and \
|
|
expressions if they satisfy a heuristic notion of 'small'.";
|
|
width_heuristics: WidthHeuristics, WidthHeuristics::scaled(100), false,
|
|
"'small' heuristic values";
|
|
|
|
fn_call_width: usize, 60, true, "Maximum width of the args of a function call before \
|
|
falling back to vertical formatting.";
|
|
attr_fn_like_width: usize, 70, true, "Maximum width of the args of a function-like \
|
|
attributes before falling back to vertical formatting.";
|
|
struct_lit_width: usize, 18, true, "Maximum width in the body of a struct lit before \
|
|
falling back to vertical formatting.";
|
|
struct_variant_width: usize, 35, true, "Maximum width in the body of a struct \
|
|
variant before falling back to vertical formatting.";
|
|
array_width: usize, 60, true, "Maximum width of an array literal before falling \
|
|
back to vertical formatting.";
|
|
chain_width: usize, 60, true, "Maximum length of a chain to fit on a single line.";
|
|
single_line_if_else_max_width: usize, 50, true, "Maximum line length for single \
|
|
line if-else expressions. A value of zero means always break if-else expressions.";
|
|
single_line_let_else_max_width: usize, 50, false, "Maximum line length for single \
|
|
line let-else statements. A value of zero means always format the divergent \
|
|
`else` block over multiple lines.";
|
|
|
|
// Options that are used by the tests
|
|
stable_option: bool, false, true, "A stable option";
|
|
unstable_option: bool, false, false, "An unstable option";
|
|
partially_unstable_option: PartiallyUnstableOption, PartiallyUnstableOption::V1, true,
|
|
"A partially unstable option";
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod partially_unstable_option {
|
|
use super::{Config, PartialConfig, PartiallyUnstableOption};
|
|
use rustfmt_config_proc_macro::{nightly_only_test, stable_only_test};
|
|
use std::path::Path;
|
|
|
|
/// From the config file, we can fill with a stable variant
|
|
#[test]
|
|
fn test_from_toml_stable_value() {
|
|
let toml = r#"
|
|
partially_unstable_option = "V2"
|
|
"#;
|
|
let partial_config: PartialConfig = toml::from_str(toml).unwrap();
|
|
let config = Config::default();
|
|
let config = config.fill_from_parsed_config(partial_config, Path::new(""));
|
|
assert_eq!(
|
|
config.partially_unstable_option(),
|
|
PartiallyUnstableOption::V2
|
|
);
|
|
}
|
|
|
|
/// From the config file, we cannot fill with an unstable variant (stable only)
|
|
#[stable_only_test]
|
|
#[test]
|
|
fn test_from_toml_unstable_value_on_stable() {
|
|
let toml = r#"
|
|
partially_unstable_option = "V3"
|
|
"#;
|
|
let partial_config: PartialConfig = toml::from_str(toml).unwrap();
|
|
let config = Config::default();
|
|
let config = config.fill_from_parsed_config(partial_config, Path::new(""));
|
|
assert_eq!(
|
|
config.partially_unstable_option(),
|
|
// default value from config, i.e. fill failed
|
|
PartiallyUnstableOption::V1
|
|
);
|
|
}
|
|
|
|
/// From the config file, we can fill with an unstable variant (nightly only)
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_from_toml_unstable_value_on_nightly() {
|
|
let toml = r#"
|
|
partially_unstable_option = "V3"
|
|
"#;
|
|
let partial_config: PartialConfig = toml::from_str(toml).unwrap();
|
|
let config = Config::default();
|
|
let config = config.fill_from_parsed_config(partial_config, Path::new(""));
|
|
assert_eq!(
|
|
config.partially_unstable_option(),
|
|
PartiallyUnstableOption::V3
|
|
);
|
|
}
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn test_config_set() {
|
|
let mut config = Config::default();
|
|
config.set().verbose(Verbosity::Quiet);
|
|
assert_eq!(config.verbose(), Verbosity::Quiet);
|
|
config.set().verbose(Verbosity::Normal);
|
|
assert_eq!(config.verbose(), Verbosity::Normal);
|
|
}
|
|
|
|
#[test]
|
|
fn test_config_used_to_toml() {
|
|
let config = Config::default();
|
|
|
|
let merge_derives = config.merge_derives();
|
|
let skip_children = config.skip_children();
|
|
|
|
let used_options = config.used_options();
|
|
let toml = used_options.to_toml().unwrap();
|
|
assert_eq!(
|
|
toml,
|
|
format!("merge_derives = {merge_derives}\nskip_children = {skip_children}\n",)
|
|
);
|
|
}
|
|
|
|
#[test]
|
|
fn test_was_set() {
|
|
let config = Config::from_toml("hard_tabs = true", Path::new("")).unwrap();
|
|
|
|
assert_eq!(config.was_set().hard_tabs(), true);
|
|
assert_eq!(config.was_set().verbose(), false);
|
|
}
|
|
|
|
const PRINT_DOCS_STABLE_OPTION: &str = "stable_option <boolean> Default: false";
|
|
const PRINT_DOCS_UNSTABLE_OPTION: &str = "unstable_option <boolean> Default: false (unstable)";
|
|
const PRINT_DOCS_PARTIALLY_UNSTABLE_OPTION: &str =
|
|
"partially_unstable_option [V1|V2|V3 (unstable)] Default: V1";
|
|
|
|
#[test]
|
|
fn test_print_docs_exclude_unstable() {
|
|
use self::mock::Config;
|
|
|
|
let mut output = Vec::new();
|
|
Config::print_docs(&mut output, false);
|
|
|
|
let s = str::from_utf8(&output).unwrap();
|
|
assert_eq!(s.contains(PRINT_DOCS_STABLE_OPTION), true);
|
|
assert_eq!(s.contains(PRINT_DOCS_UNSTABLE_OPTION), false);
|
|
assert_eq!(s.contains(PRINT_DOCS_PARTIALLY_UNSTABLE_OPTION), true);
|
|
}
|
|
|
|
#[test]
|
|
fn test_print_docs_include_unstable() {
|
|
use self::mock::Config;
|
|
|
|
let mut output = Vec::new();
|
|
Config::print_docs(&mut output, true);
|
|
|
|
let s = str::from_utf8(&output).unwrap();
|
|
assert_eq!(s.contains(PRINT_DOCS_STABLE_OPTION), true);
|
|
assert_eq!(s.contains(PRINT_DOCS_UNSTABLE_OPTION), true);
|
|
assert_eq!(s.contains(PRINT_DOCS_PARTIALLY_UNSTABLE_OPTION), true);
|
|
}
|
|
|
|
#[test]
|
|
fn test_dump_default_config() {
|
|
let default_config = format!(
|
|
r#"max_width = 100
|
|
hard_tabs = false
|
|
tab_spaces = 4
|
|
newline_style = "Auto"
|
|
indent_style = "Block"
|
|
use_small_heuristics = "Default"
|
|
fn_call_width = 60
|
|
attr_fn_like_width = 70
|
|
struct_lit_width = 18
|
|
struct_variant_width = 35
|
|
array_width = 60
|
|
chain_width = 60
|
|
single_line_if_else_max_width = 50
|
|
single_line_let_else_max_width = 50
|
|
wrap_comments = false
|
|
format_code_in_doc_comments = false
|
|
doc_comment_code_block_width = 100
|
|
comment_width = 80
|
|
normalize_comments = false
|
|
normalize_doc_attributes = false
|
|
format_strings = false
|
|
format_macro_matchers = false
|
|
format_macro_bodies = true
|
|
skip_macro_invocations = []
|
|
hex_literal_case = "Preserve"
|
|
empty_item_single_line = true
|
|
struct_lit_single_line = true
|
|
fn_single_line = false
|
|
where_single_line = false
|
|
imports_indent = "Block"
|
|
imports_layout = "Mixed"
|
|
imports_granularity = "Preserve"
|
|
group_imports = "Preserve"
|
|
reorder_imports = true
|
|
reorder_modules = true
|
|
reorder_impl_items = false
|
|
type_punctuation_density = "Wide"
|
|
space_before_colon = false
|
|
space_after_colon = true
|
|
spaces_around_ranges = false
|
|
binop_separator = "Front"
|
|
remove_nested_parens = true
|
|
combine_control_expr = true
|
|
short_array_element_width_threshold = 10
|
|
overflow_delimited_expr = false
|
|
struct_field_align_threshold = 0
|
|
enum_discrim_align_threshold = 0
|
|
match_arm_blocks = true
|
|
match_arm_leading_pipes = "Never"
|
|
force_multiline_blocks = false
|
|
fn_params_layout = "Tall"
|
|
brace_style = "SameLineWhere"
|
|
control_brace_style = "AlwaysSameLine"
|
|
trailing_semicolon = true
|
|
trailing_comma = "Vertical"
|
|
match_block_trailing_comma = false
|
|
blank_lines_upper_bound = 1
|
|
blank_lines_lower_bound = 0
|
|
edition = "2015"
|
|
version = "One"
|
|
inline_attribute_width = 0
|
|
format_generated_files = true
|
|
merge_derives = true
|
|
use_try_shorthand = false
|
|
use_field_init_shorthand = false
|
|
force_explicit_abi = true
|
|
condense_wildcard_suffixes = false
|
|
color = "Auto"
|
|
required_version = "{}"
|
|
unstable_features = false
|
|
disable_all_formatting = false
|
|
skip_children = false
|
|
hide_parse_errors = false
|
|
error_on_line_overflow = false
|
|
error_on_unformatted = false
|
|
ignore = []
|
|
emit_mode = "Files"
|
|
make_backup = false
|
|
"#,
|
|
env!("CARGO_PKG_VERSION")
|
|
);
|
|
let toml = Config::default().all_options().to_toml().unwrap();
|
|
assert_eq!(&toml, &default_config);
|
|
}
|
|
|
|
#[stable_only_test]
|
|
#[test]
|
|
fn test_as_not_nightly_channel() {
|
|
let mut config = Config::default();
|
|
assert_eq!(config.was_set().unstable_features(), false);
|
|
config.set().unstable_features(true);
|
|
assert_eq!(config.was_set().unstable_features(), false);
|
|
}
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_as_nightly_channel() {
|
|
let mut config = Config::default();
|
|
config.set().unstable_features(true);
|
|
// When we don't set the config from toml or command line options it
|
|
// doesn't get marked as set by the user.
|
|
assert_eq!(config.was_set().unstable_features(), false);
|
|
config.set().unstable_features(true);
|
|
assert_eq!(config.unstable_features(), true);
|
|
}
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_unstable_from_toml() {
|
|
let config = Config::from_toml("unstable_features = true", Path::new("")).unwrap();
|
|
assert_eq!(config.was_set().unstable_features(), true);
|
|
assert_eq!(config.unstable_features(), true);
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod deprecated_option_merge_imports {
|
|
use super::*;
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_old_option_set() {
|
|
let toml = r#"
|
|
unstable_features = true
|
|
merge_imports = true
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.imports_granularity(), ImportGranularity::Crate);
|
|
}
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_both_set() {
|
|
let toml = r#"
|
|
unstable_features = true
|
|
merge_imports = true
|
|
imports_granularity = "Preserve"
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.imports_granularity(), ImportGranularity::Preserve);
|
|
}
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_new_overridden() {
|
|
let toml = r#"
|
|
unstable_features = true
|
|
merge_imports = true
|
|
"#;
|
|
let mut config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
config.override_value("imports_granularity", "Preserve");
|
|
assert_eq!(config.imports_granularity(), ImportGranularity::Preserve);
|
|
}
|
|
|
|
#[nightly_only_test]
|
|
#[test]
|
|
fn test_old_overridden() {
|
|
let toml = r#"
|
|
unstable_features = true
|
|
imports_granularity = "Module"
|
|
"#;
|
|
let mut config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
config.override_value("merge_imports", "true");
|
|
// no effect: the new option always takes precedence
|
|
assert_eq!(config.imports_granularity(), ImportGranularity::Module);
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod use_small_heuristics {
|
|
use super::*;
|
|
|
|
#[test]
|
|
fn test_default_sets_correct_widths() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Default"
|
|
max_width = 200
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 120);
|
|
assert_eq!(config.attr_fn_like_width(), 140);
|
|
assert_eq!(config.chain_width(), 120);
|
|
assert_eq!(config.fn_call_width(), 120);
|
|
assert_eq!(config.single_line_if_else_max_width(), 100);
|
|
assert_eq!(config.struct_lit_width(), 36);
|
|
assert_eq!(config.struct_variant_width(), 70);
|
|
}
|
|
|
|
#[test]
|
|
fn test_max_sets_correct_widths() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Max"
|
|
max_width = 120
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 120);
|
|
assert_eq!(config.attr_fn_like_width(), 120);
|
|
assert_eq!(config.chain_width(), 120);
|
|
assert_eq!(config.fn_call_width(), 120);
|
|
assert_eq!(config.single_line_if_else_max_width(), 120);
|
|
assert_eq!(config.struct_lit_width(), 120);
|
|
assert_eq!(config.struct_variant_width(), 120);
|
|
}
|
|
|
|
#[test]
|
|
fn test_off_sets_correct_widths() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Off"
|
|
max_width = 100
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), usize::max_value());
|
|
assert_eq!(config.attr_fn_like_width(), usize::max_value());
|
|
assert_eq!(config.chain_width(), usize::max_value());
|
|
assert_eq!(config.fn_call_width(), usize::max_value());
|
|
assert_eq!(config.single_line_if_else_max_width(), 0);
|
|
assert_eq!(config.struct_lit_width(), 0);
|
|
assert_eq!(config.struct_variant_width(), 0);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_works_with_default() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Default"
|
|
array_width = 20
|
|
attr_fn_like_width = 40
|
|
chain_width = 20
|
|
fn_call_width = 90
|
|
single_line_if_else_max_width = 40
|
|
struct_lit_width = 30
|
|
struct_variant_width = 34
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 20);
|
|
assert_eq!(config.attr_fn_like_width(), 40);
|
|
assert_eq!(config.chain_width(), 20);
|
|
assert_eq!(config.fn_call_width(), 90);
|
|
assert_eq!(config.single_line_if_else_max_width(), 40);
|
|
assert_eq!(config.struct_lit_width(), 30);
|
|
assert_eq!(config.struct_variant_width(), 34);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_with_max() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Max"
|
|
array_width = 20
|
|
attr_fn_like_width = 40
|
|
chain_width = 20
|
|
fn_call_width = 90
|
|
single_line_if_else_max_width = 40
|
|
struct_lit_width = 30
|
|
struct_variant_width = 34
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 20);
|
|
assert_eq!(config.attr_fn_like_width(), 40);
|
|
assert_eq!(config.chain_width(), 20);
|
|
assert_eq!(config.fn_call_width(), 90);
|
|
assert_eq!(config.single_line_if_else_max_width(), 40);
|
|
assert_eq!(config.struct_lit_width(), 30);
|
|
assert_eq!(config.struct_variant_width(), 34);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_with_off() {
|
|
let toml = r#"
|
|
use_small_heuristics = "Off"
|
|
array_width = 20
|
|
attr_fn_like_width = 40
|
|
chain_width = 20
|
|
fn_call_width = 90
|
|
single_line_if_else_max_width = 40
|
|
struct_lit_width = 30
|
|
struct_variant_width = 34
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 20);
|
|
assert_eq!(config.attr_fn_like_width(), 40);
|
|
assert_eq!(config.chain_width(), 20);
|
|
assert_eq!(config.fn_call_width(), 90);
|
|
assert_eq!(config.single_line_if_else_max_width(), 40);
|
|
assert_eq!(config.struct_lit_width(), 30);
|
|
assert_eq!(config.struct_variant_width(), 34);
|
|
}
|
|
|
|
#[test]
|
|
fn test_fn_call_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 90
|
|
fn_call_width = 95
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.fn_call_width(), 90);
|
|
}
|
|
|
|
#[test]
|
|
fn test_attr_fn_like_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 80
|
|
attr_fn_like_width = 90
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.attr_fn_like_width(), 80);
|
|
}
|
|
|
|
#[test]
|
|
fn test_struct_lit_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 78
|
|
struct_lit_width = 90
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.struct_lit_width(), 78);
|
|
}
|
|
|
|
#[test]
|
|
fn test_struct_variant_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 80
|
|
struct_variant_width = 90
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.struct_variant_width(), 80);
|
|
}
|
|
|
|
#[test]
|
|
fn test_array_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 60
|
|
array_width = 80
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.array_width(), 60);
|
|
}
|
|
|
|
#[test]
|
|
fn test_chain_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 80
|
|
chain_width = 90
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.chain_width(), 80);
|
|
}
|
|
|
|
#[test]
|
|
fn test_single_line_if_else_max_width_config_exceeds_max_width() {
|
|
let toml = r#"
|
|
max_width = 70
|
|
single_line_if_else_max_width = 90
|
|
"#;
|
|
let config = Config::from_toml(toml, Path::new("")).unwrap();
|
|
assert_eq!(config.single_line_if_else_max_width(), 70);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_fn_call_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("fn_call_width", "101");
|
|
assert_eq!(config.fn_call_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_attr_fn_like_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("attr_fn_like_width", "101");
|
|
assert_eq!(config.attr_fn_like_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_struct_lit_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("struct_lit_width", "101");
|
|
assert_eq!(config.struct_lit_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_struct_variant_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("struct_variant_width", "101");
|
|
assert_eq!(config.struct_variant_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_array_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("array_width", "101");
|
|
assert_eq!(config.array_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_chain_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("chain_width", "101");
|
|
assert_eq!(config.chain_width(), 100);
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_single_line_if_else_max_width_exceeds_max_width() {
|
|
let mut config = Config::default();
|
|
config.override_value("single_line_if_else_max_width", "101");
|
|
assert_eq!(config.single_line_if_else_max_width(), 100);
|
|
}
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod partially_unstable_option {
|
|
use super::mock::{Config, PartiallyUnstableOption};
|
|
|
|
/// From the command line, we can override with a stable variant.
|
|
#[test]
|
|
fn test_override_stable_value() {
|
|
let mut config = Config::default();
|
|
config.override_value("partially_unstable_option", "V2");
|
|
assert_eq!(
|
|
config.partially_unstable_option(),
|
|
PartiallyUnstableOption::V2
|
|
);
|
|
}
|
|
|
|
/// From the command line, we can override with an unstable variant.
|
|
#[test]
|
|
fn test_override_unstable_value() {
|
|
let mut config = Config::default();
|
|
config.override_value("partially_unstable_option", "V3");
|
|
assert_eq!(
|
|
config.partially_unstable_option(),
|
|
PartiallyUnstableOption::V3
|
|
);
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
fn test_override_skip_macro_invocations() {
|
|
let mut config = Config::default();
|
|
config.override_value("skip_macro_invocations", r#"["*", "println"]"#);
|
|
assert_eq!(
|
|
config.skip_macro_invocations(),
|
|
MacroSelectors(vec![
|
|
MacroSelector::All,
|
|
MacroSelector::Name(MacroName::new("println".to_owned()))
|
|
])
|
|
);
|
|
}
|
|
}
|