df: use blocksize of 512 if POSIXLY_CORRECT is set

src/uu/df/src/blocks.rs

@@ -5,7 +5,7 @@
 //! Types for representing and displaying block sizes.
 use crate::{OPT_BLOCKSIZE, OPT_HUMAN_READABLE_BINARY, OPT_HUMAN_READABLE_DECIMAL};
 use clap::ArgMatches;
-use std::fmt;
+use std::{env, fmt};
 
 use uucore::parse_size::{parse_size, ParseSizeError};
 
@@ -127,6 +127,7 @@ fn to_magnitude_and_suffix(n: u128) -> Result<String, ()> {
 /// so on in the case of [`BlockSize::HumanReadableDecimal`]).
 ///
 /// The default variant is `Bytes(1024)`.
+#[derive(Debug, PartialEq)]
 pub(crate) enum BlockSize {
     /// A fixed number of bytes.
     ///
@@ -150,7 +151,11 @@ pub(crate) enum BlockSize {
 
 impl Default for BlockSize {
     fn default() -> Self {
-        Self::Bytes(1024)
+        if env::var("POSIXLY_CORRECT").is_ok() {
+            Self::Bytes(512)
+        } else {
+            Self::Bytes(1024)
+        }
     }
 }
 
@@ -183,6 +188,8 @@ impl fmt::Display for BlockSize {
 #[cfg(test)]
 mod tests {
 
+    use std::env;
+
     use crate::blocks::{to_magnitude_and_suffix, BlockSize};
 
     #[test]
@@ -238,4 +245,12 @@ mod tests {
             "3M-blocks"
         );
     }
+
+    #[test]
+    fn test_default_block_size() {
+        assert_eq!(BlockSize::Bytes(1024), BlockSize::default());
+        env::set_var("POSIXLY_CORRECT", "1");
+        assert_eq!(BlockSize::Bytes(512), BlockSize::default());
+        env::remove_var("POSIXLY_CORRECT");
+    }
 }

tests/by-util/test_df.rs

@@ -375,6 +375,26 @@ fn test_iuse_percentage() {
     }
 }
 
+#[test]
+fn test_default_block_size() {
+    let output = new_ucmd!()
+        .arg("--output=size")
+        .succeeds()
+        .stdout_move_str();
+    let header = output.lines().next().unwrap().to_string();
+
+    assert_eq!(header, "1K-blocks");
+
+    let output = new_ucmd!()
+        .arg("--output=size")
+        .env("POSIXLY_CORRECT", "1")
+        .succeeds()
+        .stdout_move_str();
+    let header = output.lines().next().unwrap().to_string();
+
+    assert_eq!(header, "512B-blocks");
+}
+
 #[test]
 fn test_block_size_1024() {
     fn get_header(block_size: u64) -> String {