Remove old test-cases code

src/fadt.rs

@@ -103,93 +103,3 @@ where
 
     Ok(())
 }
-
-#[cfg(test)]
-mod tests {
-    use fadt::Fadt;
-    use sdt::SdtHeader;
-    use std::mem;
-    use GenericAddress;
-
-    impl Fadt {
-        fn make_testcase(
-            oem_id: [u8; 6],
-            oem_table_id: [u8; 8],
-            oem_revision: u32,
-            creator_id: u32,
-            creator_revision: u32,
-        ) -> Fadt {
-            Fadt {
-                header: SdtHeader::make_testcase(
-                    *b"FACP",
-                    mem::size_of::<Fadt>() as u32,
-                    6,
-                    5, //checksum
-                    oem_id,
-                    oem_table_id,
-                    oem_revision,
-                    creator_id,
-                    creator_revision,
-                ),
-                firmware_ctrl: 0xDEADBEEF as u32,
-                dsdt_address: 0xDEADBEEF as u32,
-
-                // used in acpi 1.0; compatibility only, should be zero
-                reserved: 0 as u8,
-
-                preferred_pm_profile: 0 as u8,
-                sci_interrupt: 0 as u16,
-                smi_cmd_port: 0 as u32,
-                acpi_enable: 0 as u8,
-                acpi_disable: 0 as u8,
-                s4bios_req: 0 as u8,
-                pstate_control: 0 as u8,
-                pm1a_event_block: 0xDEADBEEF as u32,
-                pm1b_event_block: 0xDEADBEEF as u32,
-                pm1a_control_block: 0xDEADBEEF as u32,
-                pm1b_control_block: 0xDEADBEEF as u32,
-                pm2_control_block: 0xDEADBEEF as u32,
-                pm_timer_block: 0xDEADBEEF as u32,
-                gpe0_block: 0xDEADBEEF as u32,
-                gpe1_block: 0xDEADBEEF as u32,
-                pm1_event_length: 4 as u8,
-                pm1_control_length: 2 as u8,
-                pm2_control_length: 0 as u8,
-                pm_timer_length: 0 as u8,
-                gpe0_block_length: 2 as u8,
-                gpe1_block_length: 2 as u8,
-                gpe1_base: 0 as u8,
-                c_state_control: 0 as u8,
-                worst_c2_latency: 0 as u16,
-                worst_c3_latency: 0 as u16,
-                flush_size: 0 as u16,
-                flush_stride: 0 as u16,
-                duty_offset: 0 as u8,
-                duty_width: 0 as u8,
-                day_alarm: 0 as u8,
-                month_alarm: 0 as u8,
-                century: 0 as u8,
-                iapc_boot_arch: 0 as u16,
-                reserved2: 0 as u8,
-                flags: 0 as u32,
-                reset_reg: GenericAddress::make_testcase(),
-                reset_value: 0 as u8,
-                arm_boot_arch: 0 as u16,
-                fadt_minor_version: 2 as u8,
-                x_firmware_control: 0 as u64,
-                x_dsdt_address: 0 as u64,
-                x_pm1a_event_block: GenericAddress::make_testcase(),
-                x_pm1b_event_block: GenericAddress::make_testcase(),
-                x_pm1a_control_block: GenericAddress::make_testcase(),
-                x_pm1b_control_block: GenericAddress::make_testcase(),
-                x_pm2_control_block: GenericAddress::make_testcase(),
-                x_pm_timer_block: GenericAddress::make_testcase(),
-                x_gpe0_block: GenericAddress::make_testcase(),
-                x_gpe1_block: GenericAddress::make_testcase(),
-                sleep_control_reg: GenericAddress::make_testcase(),
-                sleep_status_reg: GenericAddress::make_testcase(),
-                hypervisor_vendor_id: 0 as u64,
-            }
-        }
-    }
-}

src/hpet.rs

@@ -17,40 +17,3 @@ pub fn parse_hpet(mapping: &PhysicalMapping<Hpet>) -> Result<(), AcpiError> {
 
     Ok(())
 }
-
-#[cfg(test)]
-mod tests {
-    use hpet::Hpet;
-    use sdt::SdtHeader;
-    use std::mem;
-    use GenericAddress;
-
-    impl Hpet {
-        fn make_testcase(
-            oem_id: [u8; 6],
-            oem_table_id: [u8; 8],
-            oem_revision: u32,
-            creator_id: u32,
-            creator_revision: u32,
-        ) -> Hpet {
-            Hpet {
-                header: SdtHeader::make_testcase(
-                    *b"HPET",
-                    mem::size_of::<Hpet>() as u32,
-                    1,
-                    3, //checksum
-                    oem_id,
-                    oem_table_id,
-                    oem_revision,
-                    creator_id,
-                    creator_revision,
-                ),
-                event_timer_block_id: 0,
-                base_address: GenericAddress::make_testcase(),
-                hpet_number: 0,
-                clock_tick_unit: 0,
-                page_protection_oem: 0,
-            }
-        }
-    }
-}

src/lib.rs

@@ -4,7 +4,6 @@
 #![feature(exclusive_range_pattern, range_contains)]
 #![feature(exhaustive_integer_patterns)]
 
-#[cfg(test)]
 #[macro_use]
 extern crate std;
 
@@ -286,20 +285,3 @@ where
     handler.unmap_physical_region(mapping);
     Ok(acpi)
 }
-
-#[cfg(test)]
-mod tests {
-    use GenericAddress;
-
-    impl GenericAddress {
-        pub(crate) fn make_testcase() -> GenericAddress {
-            GenericAddress {
-                address_space: 0 as u8,
-                bit_width: 0 as u8,
-                bit_offset: 0 as u8,
-                access_size: 0 as u8,
-                address: 0 as u64,
-            }
-        }
-    }
-}

src/rsdp.rs

@@ -88,53 +88,3 @@ impl Rsdp {
         self.xsdt_address
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use rsdp::Rsdp;
-
-    impl Rsdp {
-        /// Create a test RSDP. Checksums are passed as `Option<u8>`; if `None` is passed, the
-        /// correct checksum is calculated and used.
-        pub fn make_testcase(
-            signature: [u8; 8],
-            checksum: Option<u8>,
-            oem_id: [u8; 6],
-            revision: u8,
-            rsdt_address: u32,
-            length: u32,
-            xsdt_address: u64,
-            ext_checksum: Option<u8>,
-            reserved: [u8; 3],
-        ) -> Rsdp {
-            let checksum = checksum.unwrap_or(
-                ((0isize
-                    - signature.iter().map(|&b| isize::from(b)).sum::<isize>()
-                    - oem_id.iter().map(|&b| isize::from(b)).sum::<isize>()
-                    - revision as isize
-                    - rsdt_address as isize)
-                    & 0b1111_1111) as u8,
-            );
-
-            let ext_checksum = ext_checksum.unwrap_or(
-                ((0isize
-                    - length as isize
-                    - xsdt_address as isize
-                    - reserved.iter().map(|&b| isize::from(b)).sum::<isize>())
-                    & 0b1111_1111) as u8,
-            );
-
-            Rsdp {
-                signature,
-                checksum,
-                oem_id,
-                revision,
-                rsdt_address,
-                length,
-                xsdt_address,
-                ext_checksum,
-                reserved,
-            }
-        }
-    }
-}

src/sdt.rs

@@ -113,31 +113,6 @@ impl SdtHeader {
         // Safe to unwrap because checked in `validate`
         str::from_utf8(&self.oem_table_id).unwrap()
     }
-
-    #[cfg(test)]
-    pub(crate) fn make_testcase(
-        signature: [u8; 4],
-        length: u32,
-        revision: u8,
-        checksum: u8,
-        oem_id: [u8; 6],
-        oem_table_id: [u8; 8],
-        oem_revision: u32,
-        creator_id: u32,
-        creator_revision: u32,
-    ) -> SdtHeader {
-        SdtHeader {
-            signature,
-            length,
-            revision,
-            checksum,
-            oem_id,
-            oem_table_id,
-            oem_revision,
-            creator_id,
-            creator_revision,
-        }
-    }
 }
 
 /// Takes the physical address of an SDT, and maps, clones and unmaps its header. Useful for